nl-design-system / candidate

NL Design System Candidate components
https://nl-design-system.github.io/candidate/
European Union Public License 1.2
0 stars 0 forks source link

Link - Community Utrecht - A11y getest #199

Closed rianrietveld closed 4 days ago

rianrietveld commented 1 week ago

Review toegankelijkheid voor de Link component in Community bij Utrecht

Hoe te testen

Ga naar de te reviewen component in StoryBook

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

Toegankelijkheid visueel ontwerp

Toegankelijkheid zoom en herschalen

Toegankelijkheid toetsenbord

Toegankelijkheid screenreader

MarjonBakker commented 1 week ago

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:

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 commented 1 week ago

@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.