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

Code - Community Utrecht - A11y getest #193

Closed
6 tasks done
rianrietveld opened this issue Nov 19, 2024 · 3 comments
Closed
6 tasks done

Code - Community Utrecht - A11y getest #193

rianrietveld opened this issue Nov 19, 2024 · 3 comments
Labels
accessibility Ook wel bekend als "a11y"
Milestone

Comments

@rianrietveld
Copy link
Contributor

rianrietveld commented Nov 19, 2024

Review toegankelijkheid voor de Code component in Community bij Utrecht

Note: Amsterdam heeft geen Community component voor code.

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 Rian op 20 november 2024

Gereviewed: Code in StoryBook Utrecht
Visueel en via de "Show code"-optie bij de voorbeelden.

HTML basisvoorbeeld:

<code class="utrecht-code">
  &lt;input type=&quot;url&quot; value=&quot;https://example.fi/&quot;&gt;
</code>;

Bij de voorbeelden in dit Storybook is de CSS voor code niet gebruikt.

.utrecht-code {
  background-color: var(--utrecht-code-background-color);
  color: var(--utrecht-code-color);
  font-family: var(--utrecht-code-font-family, monospace),monospace;
  font-size: var(--utrecht-code-font-size, inherit);
  font-variant-ligatures: none;
  line-height: var(--utrecht-code-line-height);
}

De fallback voor font-family is dus monospace.
Ik ben van de review uitgegaan van deze CSS-loze component.

Er is de mogelijkheid om ook <span> te gebruiken bij de keuze van het element. Deze review geldt alleen wanneer het element <code> wordt gekozen.

Acceptatiecriteria

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

Toegankelijkheid visueel ontwerp

  • De tekst, gemarkeerd als code, is niet alleen herkenbaar aan een andere achtergrondkleur.
    WCAG 1.4.1 Gebruik van kleur
    Oordeel: Voldoet. De tekst is te onderscheiden door het mono-space lettertype.
  • De tekst, gemarkeerd als code, heeft voldoende contrast met de achtergrondkleur.
    WCAG 1.4.3 Contrast (minimum)
    Oordeel: Voldoet.
  • Wanneer de achtergrondkleur van de Code component anders is dan de default browserkleur, dan heeft deze een kleurcontrast van minimaal 3:1 met de achtergrondkleur van de pagina.
    WCAG 1.4.11 Contrast van niet-tekstuele content
    Oordeel: Voldoet.

Toegankelijkheid zoom en herschalen

Toegankelijkheid screenreader

@rianrietveld rianrietveld converted this from a draft issue Nov 19, 2024
@rianrietveld rianrietveld added the accessibility Ook wel bekend als "a11y" label Nov 19, 2024
@rianrietveld rianrietveld changed the title Code -Amsterdam - A11y getest Code - Amsterdam - A11y getest Nov 19, 2024
@rianrietveld rianrietveld changed the title Code - Amsterdam - A11y getest Code - Utrecht - A11y getest Nov 19, 2024
@rianrietveld rianrietveld modified the milestone: Code Nov 19, 2024
@rianrietveld rianrietveld changed the title Code - Utrecht - A11y getest Code - Community Utrecht - A11y getest Nov 20, 2024
@MarjonBakker
Copy link
Contributor

@rianrietveld je hebt deze er dubbel in staan: "Wanneer de achtergrondkleur van de Code component anders is dan de default browserkleur, dan heeft deze een kleurcontrast van minimaal 3:1 met de achtergrondkleur van de pagina."

Een keer als 'voldoet' en een keer als 'niet van toepassing'.

@MarjonBakker MarjonBakker removed their assignment Nov 21, 2024
@rianrietveld
Copy link
Contributor Author

@MarjonBakker Dank!
Het is aangepast en ik heb er ook de links naar de WCAG-succescriteria bijgezet.

@rianrietveld
Copy link
Contributor Author

@matijs @jeffreylauwers
Code component van Utrecht is a11y approved, we geven um aan jullie door.

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