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(TextInput, InvalidField): remove a11y attribute error message + aria-invalid is automatic #1034

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

LarryMatte
Copy link
Contributor

DS-1257

Les messages d'erreur dans le design system utilisaient des notifications automatiques pour les lecteurs d'écran (aria-live et role="alert"), mais avec le nouveau pattern de validation des formulaires, ces notifications ne sont plus nécessaires. De plus, l'attribut aria-invalid est maintenant ajouté dès que le champ est en erreur.

@LarryMatte LarryMatte requested a review from a team as a code owner November 14, 2024 16:48
Copy link

Storybook for this build: https://ds.equisoft.io/pr-1034/

Copy link

Webapp for this build: https://ds.equisoft.io/pr-1034/webapp/

@meriouma
Copy link
Contributor

meriouma commented Dec 5, 2024

Est-ce que tu voulais le mettre sur TextInput seulement (et ceux qui l'utilisent), ou tous les inputs? Parce que c'est pas tout les inputs pour l'instant qui utilisent le TextInput, alors je suis pas certain que le aria-invalid se ramasse partout

@LarryMatte
Copy link
Contributor Author

Est-ce que tu voulais le mettre sur TextInput seulement (et ceux qui l'utilisent), ou tous les inputs? Parce que c'est pas tout les inputs pour l'instant qui utilisent le TextInput, alors je suis pas certain que le aria-invalid se ramasse partout

L'aria-invalid était déjà dans le composant. J'ai seulement modifié pour qu'il soit affiché à true au moment où le composant est invalid. Pour les autres composants, nous ferons une passe après que le travail de simplification des inputs sera fait, i.e., avoir un input core et l'utiliser dans les autres inputs, ca va éviter le travail en double.

@LarryMatte LarryMatte merged commit 45dc6fc into master Dec 9, 2024
22 checks passed
@LarryMatte LarryMatte deleted the dev/DS-1257 branch December 9, 2024 14:24
LarryMatte added a commit that referenced this pull request Jan 9, 2025
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