detfaellesdesignsystem / dkfds-components

Styling og komponenter
Other
44 stars 13 forks source link

Tooltip - fejl ved tilgængelighedsprøve - mangler stickyness #163

Closed JOKBODIGST closed 10 hours ago

JOKBODIGST commented 2 years ago

[Indmeld kun 1 fejl pr. issue og tjek gerne om fejlen allerede er indmeldt. (Only one bug per issue and make sure to check if the issue already exists)]

Bekskriv fejlen (Describe the bug) Vær klar og præcis i din beskrivelse af fejlen. (A clear and concise description of what the bug is.)

Tooltip. Mangler stickyness, som gør at man kan aktiverer tooltip med museklik, mellemrum eller enter og derefter tabulere videre. Der er også problemer med at man kan have flere tooltips åbne samtidig. Desuden understøtter den ikke ”svævepeg”, hvilket er at man skal kunne nå at give tooltip tekst fokus ved hover inden den forsvinder.

Genskab fejlen (To Reproduce) Sådan genskabes fejlen (Steps to reproduce the behavior):

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Fejlen er blevet indrapporteret i forbindelse med en webtilgængelighedstest.

Indsæt link (Add a URL) Tilføj om muligt et link til fejlen. (If possible, add a URL to help identify your problem.) N/A

Screenshots Tilføj om muligt et eller flere billeder der viser fejlen. (If applicable, add screenshots to help explain your problem.) N/A

Sådan burde det fungere (Expected behavior) Beskriv om muligt, hvordan fejlen løses eller beskriv klart og præcist hvordan det burde fungere. (A clear and concise description of what you expected to happen.)

Se beskivelse af fejl.

Desktop - udfyld venligst følgende (please complete the following information):

Smartphone - udfyld venligst følgende (please complete the following information):

Supplerende oplysninger (Additional context) Tilføj andre detaljer vedrørende fejlen. (Add any other context about the problem here.)

detfaellesdesignsystem commented 2 years ago

Hej @JOKBODIGST

Tak for henvendelsen.

Vi har oprettet en sag på det.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

detfaellesdesignsystem commented 2 years ago

Hej @JOKBODIGST,

Kan du uddybe lidt i forhold til, hvad den ønskede opførelse er i forhold til de meldte udfordringer med tooltip komponenten?

Særligt med hensyn til: "Mangler stickyness, som gør at man kan aktiverer tooltip med museklik, mellemrum eller enter og derefter tabulere videre."

Bemærk at tooltip komponenten blev opdateret i version 7.4.0.

For at undgå overlap af tooltips kan man kun have én tooltip åben ad gangen. Det er lidt uklart, hvad det er I ønsker her?

Toolltip er ikke et fokusérbart element, men det jeg gætter på I ønsker er at man skal kunne markere teksten for at få den læst op. Det skal selvfølgelig kunne lade sig gøre ved klik på tooltipknappen samt fokus på tooltipknappen. Hvorvidt det kan lade sig gøre ved hover på tooltip knappen skal lige vendes internt i FDS teamet.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

cjessing commented 2 years ago

WCAG 1.4.13 Indhold ved svævepegning (hover) eller fokus - Niveau AA

Når elementer modtager fokus via musen (svævemarkøren) eller tastaturet og udløser ekstra indhold/en hovertekst, skal det ekstra indhold/hoverteksten opfylde følgende tre krav 1) Kan afvises, 2) svævepegning mulig og 3) blivende.

Claus Jessing

M: +45 6061 0101 Email: @. Encrypted email: @.

On Fri, Jan 14, 2022 at 2:08 PM detfaellesdesignsystem < @.***> wrote:

Hej @JOKBODIGST https://github.com/JOKBODIGST,

Kan du uddybe lidt i forhold til, hvad den ønskede opførelse er i forhold til de meldte udfordringer med tooltip komponenten?

Særligt med hensyn til: "Mangler stickyness, som gør at man kan aktiverer tooltip med museklik, mellemrum eller enter og derefter tabulere videre."

Bemærk at tooltip komponenten blev opdateret i version 7.4.0.

For at undgå overlap af tooltips kan man kun have én tooltip åben ad gangen. Det er lidt uklart, hvad det er I ønsker her?

Toolltip er ikke et fokusérbart element, men det jeg gætter på I ønsker er at man skal kunne markere teksten for at få den læst op. Det skal selvfølgelig kunne lade sig gøre ved klik på tooltipknappen samt fokus på tooltipknappen. Hvorvidt det kan lade sig gøre ved hover på tooltip knappen skal lige vendes internt i FDS teamet.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

— Reply to this email directly, view it on GitHub https://github.com/detfaellesdesignsystem/dkfds-components/issues/163#issuecomment-1013102540, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACDLS6P3QD6Q2H6VDHLFUUDUWAN5BANCNFSM5LG3NBUQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you are subscribed to this thread.Message ID: @.*** com>

detfaellesdesignsystem commented 2 years ago

Hej @cjessing,

Tak for svar på sagen. Vi har rettet tooltip komponenten til så man kan markere tekst ved både fokus/klik på tooltipknappen samt ved hover. Tooltippet bliver derved også ved "svævepeg".

Ændringen vil komme med i næste release.

@JOKBODIGST - Med hensyn til de 2 andre meldte udfordringer er det fortsat uklart, hvad den ønskede opførelse for komponenten er:

  1. "Mangler stickyness, som gør at man kan aktiverer tooltip med museklik, mellemrum eller enter og derefter tabulere videre."
  2. "Der er også problemer med at man kan have flere tooltips åbne samtidig."

I må meget gerne uddybe, hvad den ønskede opførelse for komponenten er, og argumenter herfor, hvis det skal ændres.

Vi venter med release indtil de sidste 2 udeståender er afklaret.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

detfaellesdesignsystem commented 10 hours ago

Vi lukker tråden her, da der ikke er kommet flere kommentarer.

Der blev releaset en ny version af tooltip i version 10.0.0.