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

Paragraph - Candidate NLDS - A11y getest #213

Open
rianrietveld opened this issue Nov 20, 2024 · 2 comments · Fixed by #372
Open

Paragraph - Candidate NLDS - A11y getest #213

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

Comments

@rianrietveld
Copy link
Contributor

rianrietveld commented Nov 20, 2024

Review toegankelijkheid voor de NLDS Candidate component Paragraph

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 19-12-2024 door Marjon

Er is nog een hertest nodig met "Page Titled" en "Language of the Page" in scope, zie PR #382

Gereviewd:

Review ten opzichte van de acceptatiecriteria:

Gebruik het p-element voor paragraaftekst

Als een paragraaftekst in een andere taal is dan de taal van de pagina, dan heeft het element een lang-attribuut met de juiste taalcode

De contrastverhouding van de tekstkleur van de paragraaf met de achtergrondkleur is hoog genoeg

Als je de paragraaftekst vergroot tot 200% blijft deze in zijn geheel zichtbaar

Als je de hele webpagina inzoomt tot 400% blijft de paragraaftekst leesbaar

Als je de tekstafstand vergroot blijft de tekst in zijn geheel zichtbaar

Review toegankelijkheid 12-12-2024 door Marjon

Zie issues: #373

Zie uikomst: https://github.com/nl-design-system/candidate/blob/main/packages/storybook-test/stories/paragraph/paragraph.stories.tsx

@rianrietveld rianrietveld added the accessibility Ook wel bekend als "a11y" label Nov 20, 2024
@rianrietveld rianrietveld added this to the Paragraph milestone Nov 20, 2024
@Robbert
Copy link
Member

Robbert commented Dec 12, 2024

@MarjonBakker
Copy link
Contributor

MarjonBakker commented Dec 12, 2024

Getest. Zie #373 voor issues. PR: #372

@MarjonBakker MarjonBakker linked a pull request Dec 12, 2024 that will close this issue
@MarjonBakker MarjonBakker reopened this Dec 19, 2024
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

Successfully merging a pull request may close this issue.

4 participants