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

Link - Community Utrecht - A11y getest #199

Closed
15 of 17 tasks
rianrietveld opened this issue Nov 20, 2024 · 2 comments
Closed
15 of 17 tasks

Link - Community Utrecht - A11y getest #199

rianrietveld opened this issue Nov 20, 2024 · 2 comments
Assignees
Labels
accessibility Ook wel bekend als "a11y"
Milestone

Comments

@rianrietveld
Copy link
Contributor

rianrietveld commented Nov 20, 2024

Review toegankelijkheid voor de Link component in Community bij Utrecht

Hoe te testen

Ga naar de te reviewen component in StoryBook

  • Bij de component, kies in de zijbalk de view Default
  • Klik op het open nieuwe tab icon rechtsboven in de taakbalk.
  • Je ziet nu alleen de component met de bijbehorende CSS.

Review toegankelijkheid door Marjon op 21 november 2024

Gereviewd:

HTML basisvoorbeeld:

<a href="https://www.example.com" class="utrecht-link utrecht-link--html-a">Read more</a>

Acceptatiecriteria

Samenvatting: de Link component in Community bij Utrecht voldoet niet aan de acceptatiecriteria voor toegankelijkheid van het NL Design System.

Toegankelijkheid algemeen

  • Het label van de Link bestaat uit gewone tekst, niet uit een afbeelding van tekst
    WCAG 1.4.5 Afbeeldingen van tekst
    Oordeel: Voldoet.
  • De linktekst van de Link vertelt eenduidig aan de gebruiker waar de link naar toe gaat (het linkdoel).
    WCAG 2.4.4 Linkdoel (in context)
    Oordeel: Niet van toepassing, contextafhankelijk.
  • De zichtbare naam van de Link komt voor in de toegankelijke naam
    WCAG 2.5.3 Label in naam
    Oordeel: Voldoet.
  • Als de tekst van de Link in een andere taal is dan de taal van de pagina, dan heeft het element een lang-attribuut met de juiste taalcode
    WCAG 3.1.2 Taal van onderdelen
    Oordeel: Niet van toepassing, contextafhankelijk.

Toegankelijkheid visueel ontwerp

Toegankelijkheid zoom en herschalen

  • Als je de tekst vergroot tot 200% blijft deze in zijn geheel zichtbaar
    WCAG 1.4.4 Herschalen van tekst
    Oordeel: Voldoet.
  • Als je de tekstafstand vergroot blijft de tekst in zijn geheel zichtbaar
    WCAG 1.4.12 Tekstafstand
    Oordeel: Voldoet.

Toegankelijkheid toetsenbord

Toegankelijkheid screenreader

  • Als het label van de Link uit een icoon bestaat, dan heeft deze een goed tekstalternatief
    WCAG 1.1.1 Niet-tekstuele content
    Oordeel: Niet van toepassing.
  • De Link heeft een rol van link en een toegankelijke naam die duidelijk maakt waar de Link voor dient
    WCAG 4.1.2 Naam, rol, waarde
    Oordeel: Voldoet.
@rianrietveld rianrietveld added the accessibility Ook wel bekend als "a11y" label Nov 20, 2024
@rianrietveld rianrietveld moved this to In Progress in Candidate component taken Nov 20, 2024
@rianrietveld rianrietveld added this to the Link milestone Nov 20, 2024
@rianrietveld rianrietveld changed the title Link - Utrecht - A11y getest Link - Community Utrecht - A11y getest Nov 20, 2024
@MarjonBakker MarjonBakker moved this from In Progress to Review needed in Candidate component taken Nov 21, 2024
@MarjonBakker
Copy link
Contributor

MarjonBakker commented Nov 22, 2024

De Link component faalt misschien 2.5.5 Grootte van het aanwijsgebied.

Als de link in een paragraaf met tekst zou staan, zou er een uitzondering gelden voor dit criterium. En er is ook een uitzondering voor als de author de hoogte van de link niet heeft gedefinieerd.

Het is dus niet met zekerheid te zeggen of dit component voldoet aan de acceptatiecriteria.

Zoals @rianrietveld en ik het nu zien zouden er twee componenten/css-classes moeten komen:

  • link in tekst
  • link stand-alone

Want om een target groot genoeg te maken moet je de margin/padding aanpassen en dat kan alleen op een block of line-block element.

@MarjonBakker MarjonBakker moved this from Review needed to Done in Candidate component taken Nov 22, 2024
@MarjonBakker MarjonBakker closed this as completed by moving to Done in Candidate component taken Nov 22, 2024
@MarjonBakker
Copy link
Contributor

@jeffreylauwers @matijs Ik wil deze graag aan jullie doorgeven. Hij is niet volledig a11y goedgekeurd, daarvoor moet denk ik iets verzonnen worden om aan 2.5.5 Grootte van het aanwijsgebied te voldoen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Ook wel bekend als "a11y"
Projects
Status: Done
Development

No branches or pull requests

2 participants