SpareBank1 / designsystem

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

Fikse feil ved fontskalering i PrimaryButton, SecondaryButton, ActionButton, ShortcutButton og ExpandButton #1711

Closed andrea-sb1 closed 8 months ago

andrea-sb1 commented 11 months ago

Lovkrav

https://www.uutilsynet.no/wcag-standarden/144-endring-av-tekststorrelse-niva-aa/96

Tester

https://github.com/sparebank1utvikling/designsystem-uustatus/blob/main/reports/Buttons/ActionButton/wcag-1.4.4_endring_av_tekstst_rrelse.yml

https://github.com/sparebank1utvikling/designsystem-uustatus/blob/main/reports/Buttons/ExpandButton/wcag-1.4.4_endring_av_tekstst_rrelse.yml

https://github.com/sparebank1utvikling/designsystem-uustatus/tree/main/reports/Buttons/PrimaryButton

https://github.com/sparebank1utvikling/designsystem-uustatus/blob/main/reports/Buttons/SecondaryButton/wcag-1.4.4_endring_av_tekstst_rrelse.yml

https://github.com/sparebank1utvikling/designsystem-uustatus/blob/main/reports/Buttons/ShortcutButton/wcag-1.4.4_endring_av_tekstst_rrelse.yml

Beskrivelse

Ved skalering av tekststørrelse til 200%, i mobilvisning, blir deler av teksten (og ikonene) i knappene kuttet så det ikke er synlig. Det er testet med tekstmengder som er vanlige ved bruk av knappene i nettbanken og mobilbanken. Ved en viss lengde kuttes tekst og ikoner.

Det gjelder PrimaryButton, SecondaryButton, ActionButton, ShortcutButton og ExpandButton.

PrimaryButton med standard tekststørrelse:

PrimaryButton med tekst standard størrelse

PrimaryButton med tekststørrelse 200%:

PrimaryButton med tekst 200 prosent størrelse

Det er også mulig for kunder/brukere å velge enda større tekststørrelse enn 200% (200% er bare minimumskravet, i loven). Det vil bare forsterke problemet.

Forslag til løsning

PrimaryButton med tekst 200 prosent størrelse og foreslåtte endringer

Vi kan vurdere å endre overflow fra hidden til visible i tillegg?

antidecaf commented 8 months ago

Fikset i #1807