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

feat(ffe-form-react): add role="alert" to ErrorFieldMessage #943

Merged
merged 1 commit into from
Oct 2, 2020

Conversation

torkjels
Copy link
Contributor

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.

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.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 84.697% when pulling ff6ea6c on error-field-message-alert into c0d6d59 on develop.

@kwltrs
Copy link
Contributor

kwltrs commented Sep 28, 2020

Burde vi ha role=alert@sb1/ffe-message-box-react?

@torkjels
Copy link
Contributor Author

Burde vi ha role=alert@sb1/ffe-message-box-react?

@kwltrs Godt forslag, jeg lagde PR på ffe-message-box-react ( #944 ), ffe-content-message-react ( #945 ) og ffe-system-message-react ( #946 ).

@ivarni
Copy link
Contributor

ivarni commented Sep 28, 2020

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.

@torkjels
Copy link
Contributor Author

torkjels commented Sep 28, 2020

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 display: none). Jeg vil absolutt tro det også gjelder i alle andre tilfeller hvor skjermleser uansett ikke hadde lest opp elementet :) Så hvis man har skjult elementet riktig (enten vha. display: none eller feks ved conditional render) så leses ikke alert'en opp.

Copy link
Contributor

@pethel pethel left a 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?

@torkjels torkjels merged commit 2a84ec8 into develop Oct 2, 2020
@torkjels torkjels deleted the error-field-message-alert branch October 2, 2020 08:28
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.

6 participants