SpareBank1 / designsystem

SpareBank 1's design system and component library.
https://design.sparebank1.no
MIT License
100 stars 87 forks source link

Bytte om på aria-expanded="true"/"false" for Tooltip external #1587

Closed andrea-sb1 closed 8 months ago

andrea-sb1 commented 1 year ago

Test

https://github.com/sparebank1utvikling/designsystem-uustatus/blob/main/reports/Form/Tooltip/wcag-4.1.2_navn_rolle_verdi.yml

Beskrivelse

For external-varianten av Tooltip leser skjermlesere opp knappen som minimert når teksten vises (pga. aria-expanded="false") og utvidet når teksten er skjult (pga. aria-expanded="true"). Det blir da forvirrende å bruke akkurat den varianten med skjermleser. Det må også fikses for at komponenten skal ivareta WCAG-krav 4.1.2 Navn, rolle, verdi (Nivå A).

Skjermbilde av Tooltip external med skjermleser

Forslag

Bytte til aria-expanded="true" når teksten vises, og "false" når den er skjult :)

HeleneKassandra commented 1 year ago

Den setter allerede aria-expanded="true"/"false" etter om den er åpen eller lukket, er problemet at aria-expanded ligger på feil tag? Nå ligger aria-expanded på knappen, som også har en aria-controls med id til "collapse"

Edit: Så nå at denne feilen kun ser ut til å dukke opp i tooltip-external scenarioet. Det skyldes at man ikke sender med isOpen propertyen i eksempelet. Litt usikker på hva bruksscenarioet for den external versjonen faktisk er, og om vi burde gjøre isOpen propertyen obligatorisk på tooltip eller ikke