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

Field required indication on label instead of group #1203

Closed
MartinWeb opened this issue Sep 18, 2024 · 0 comments · Fixed by #1204
Closed

Field required indication on label instead of group #1203

MartinWeb opened this issue Sep 18, 2024 · 0 comments · Fixed by #1204
Labels
Type: Bug Design graphics bugs
Milestone

Comments

@MartinWeb
Copy link
Contributor

When an input has the required classModifier, it adds an asterix next to the label to tell that the field is required.
Right now, this is handled with the className of the group who englobes the label.
image

Unfortunately this can lead to some problems when a ancestor in the DOM has that required property and in term put some input as required (visually) when they are not supposed to.

Exemple :
image

@MartinWeb MartinWeb added the Type: Bug Design graphics bugs label Sep 18, 2024
@MartinWeb MartinWeb added this to the V3.0.0 milestone Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Design graphics bugs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant