-
Notifications
You must be signed in to change notification settings - Fork 85
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
feat(ffe-form-react): add role="alert" to ErrorFieldMessage #943
Conversation
This allows screen reader users to see a form error message immediately when it appears, in the same way that a user not using a screen reader would see it immediately.
Burde vi ha |
Fint om dere da også ber designerne bare bruke de komponentene dersom de ønsker den oppførselen. Jeg lurer også på hvordan role alert fungerer på feks skjulte elementer, siden hos oss er det populært med disse message-boxene i ekspanderbare tooltips. |
Ifølge eksempelene til MDN leses ikke alert'en opp hvis elementet er skjult (for eksempel med |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Setter needs work med samma kommentar som på dom andra. Kanske er tryggast og gjøra det slik eller vad tenker dere?
Beskrivelse
Bruker samme fremgangsmåte som i eksempel #2 på MDN sin side Using the alert role
Motivasjon og kontekst
Dette gjør at skjermleserbrukere ser skjemafeilmeldinger med én gang de oppstår, på samme måte som en "seende" bruker ser de med én gang.
Testing
Fyrte opp designsystemet på lokal maskin og testet med Talkback på Android og Orca på Firefox.