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

fix(accessibility): add attribute required based on classModifier #1045

Merged
merged 4 commits into from
Sep 12, 2023

Conversation

MartinWeb
Copy link
Contributor

Related issue

#1044

Person(s) for reviewing proposed changes

@arnaudforaison @samuel-gomez @buddyvegas

JLou
JLou previously approved these changes Sep 5, 2023
@MartinWeb MartinWeb added this to the v2.1.0 milestone Sep 6, 2023
@MartinWeb MartinWeb dismissed stale reviews from samuel-gomez-axa and JLou via d77682e September 7, 2023 14:42
@MartinWeb MartinWeb force-pushed the fix/requiredAttribute branch from f7031f1 to d77682e Compare September 7, 2023 14:42
JLou
JLou previously approved these changes Sep 7, 2023
@MartinWeb
Copy link
Contributor Author

à retester sur site, en local le aria-required du SelectMulti n'est plus présent

JLou
JLou previously approved these changes Sep 11, 2023
@MartinWeb MartinWeb force-pushed the fix/requiredAttribute branch from 8a50ed4 to 928c0bf Compare September 12, 2023 07:11
@MartinWeb MartinWeb dismissed stale reviews from samuel-gomez-axa and JLou via 19842a5 September 12, 2023 07:58
@MartinWeb MartinWeb force-pushed the fix/requiredAttribute branch from 928c0bf to 19842a5 Compare September 12, 2023 07:58
@MartinWeb MartinWeb force-pushed the fix/requiredAttribute branch from 19842a5 to b08a551 Compare September 12, 2023 08:09
@MartinWeb MartinWeb linked an issue Sep 12, 2023 that may be closed by this pull request
@MartinWeb MartinWeb merged commit 545e2c6 into AxaFrance:master Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Form] required inputs does not have the required attribute
3 participants