Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🎨 [#1139] login-screen validation-text #479

Merged
merged 2 commits into from
Feb 21, 2023

Conversation

jiromaykin
Copy link
Contributor

issue with overflowing UL's in registration view:
Taiga: https://taiga.maykinmedia.nl/project/open-inwoner/task/1139

@jiromaykin
Copy link
Contributor Author

jiromaykin commented Feb 17, 2023

I made the style specific to the ID because there was a very dominant style that changes everything to width=100%
(.form__control>.label :not(.button):not(.flatpickr-clear))

Can be tested here: http://127.0.0.1:8000/accounts/register/

@jiromaykin jiromaykin marked this pull request as ready for review February 20, 2023 08:44

/// Registration.

#registration-form .form__control label.label ul {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC there was some sort of rule/method to not use element names in the selectors, something with specificity (hence why everything has class names matching the tags).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Bartvaderkin I've changed it now - it's a unique situation that i think only occurs on the Registration page. + it's not valid HTML to have a UL tag (=block) inside a label (=inline) - but this is quick fix for now.

@alextreme alextreme merged commit dc2159c into develop Feb 21, 2023
@alextreme alextreme deleted the feature/1139-login-screen-validation-text branch February 21, 2023 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants