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

[Innspill til komponent]: <Tooltip> UU og Axe #3067

Open
ivoiv opened this issue Aug 1, 2024 · 1 comment · Fixed by #3094
Open

[Innspill til komponent]: <Tooltip> UU og Axe #3067

ivoiv opened this issue Aug 1, 2024 · 1 comment · Fixed by #3094

Comments

@ivoiv
Copy link

ivoiv commented Aug 1, 2024

Tilbakemelding

Tooltip setter aria-describedby på sine barn kun når den er aktiv - dvs mens man hovrer over elementet den beskriver. Det liker ikke AXE når denne komponenten ikke har noe beskrivende tekst selv.

F.eks.

<Tooltip content="Klikk her">
   <Button 
        as="a"
        href="https://www.nav.no"
        icon={<NavIkon aria-hidden />}
  </Button>
</Tooltip>

Her sier AXE at knappen ikke har beskrivende tekst, selv om den vil få beskrivelsen sin av Tooltip når man enten hovrer over den og når den får tastaturfokus.

Å sette aria-label eller title på Button eller Ikonet fikser det, men da blir det dobbelt opp med beskrivelser.
Er dette noe farlig, at det er potensielt to forskjellige beskrivelser?

Evt. kunne vi fikset det i selve Tooltip komponenten?

@KenAJoh
Copy link
Collaborator

KenAJoh commented Aug 13, 2024

Vi skal få sett på om det er noen forbedringer vi enkelt kanskje kan implementere her 👍 Men frem til da ser vi ikke noen store problemer med at Button har title i tillegg til Tooltip. Det vil kunne bli litt dobbelt opplesning for skjermeleser, men sikrer da i allefall at knappen er tilgjengelig.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants