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

Docs/baseline accessibility support #1791

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

MarjonBakker
Copy link
Contributor

Toevoegen eerste aanzet voor tekst over de baseline.

@MarjonBakker MarjonBakker self-assigned this Dec 9, 2024
Copy link

vercel bot commented Dec 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentatie ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 15, 2024 7:14pm

@MarjonBakker MarjonBakker changed the title Docs/baseline a11y Docs/baseline-a11y Dec 9, 2024
@MarjonBakker MarjonBakker linked an issue Dec 9, 2024 that may be closed by this pull request
@@ -13,3 +13,37 @@ keywords:
{/* @license CC0-1.0 */}

# Baseline toegankelijkheids-ondersteuning

NL Design System ondersteunt de twee meest recente versies van de volgende combinaties van de volgende besturingssystemen, browsers en screenreaders:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ik zou op deze pagina duidelijkheid verwachten over wat 'NL Design System' betekent in deze zin.

Ik denk dat we bedoelen 'NL Design System componenten met Hall of Fame status', en dat we niet bedoelen richtlijnen + componenten met een lagere status.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wmb zou de pagina daarom het meest logisch zijn als subpagina van 'Componenten', en niet bij WCAG, want onze beschrijving van de SC's heeft geen conformance status.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ik denk dat dit wel prima is eigenlijk, we kunnen zelf kiezen op welke punten we de baseline opnemen in de Definition of Done.

Het is namelijk ook handig als aanbestedingen van organisaties in de community kunnen verwijzen naar onze baseline in bijv. aanbestedingen

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Misschien kunnen we dat dan opnemen?

Suggested change
NL Design System ondersteunt de twee meest recente versies van de volgende combinaties van de volgende besturingssystemen, browsers en screenreaders:
Deze baseline kan worden gebruikt in NL Design System communityprojecten en voor het vaststellen van een baseline voor bijvoorbeeld het testen van individuele componenten.
Ondersteun de twee meest recente versies van de volgende combinaties van de volgende besturingssystemen, browsers en screenreaders:

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GOV.UK hierover: #1784 (comment)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Goed om daar even naar te kijken inderdaad.

Valt me op dat ook zij kiezen om geen claims te maken over wat zij ondersteunen, maar adviseren dat het lijstje op hun pagina kan (moet) worden gebruikt. Dat lijkt mij voor ons ook de enige manier.

@MarjonBakker MarjonBakker changed the title Docs/baseline-a11y Docs/baseline accessibility support Dec 9, 2024
Copy link
Member

@Robbert Robbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zou het handig zijn om kopje te maken met een tijdsopname van welke versies vandaag concreet ondersteund worden, met linkjes waar je in de toekomst de nieuwste versies kan vinden?

@rianrietveld
Copy link
Contributor

@Robbert @hidde @MarjonBakker
Ik laat de discussie aan jullie en bemoei me er niet mee.

@rianrietveld rianrietveld removed their request for review December 10, 2024 09:48
@MarjonBakker
Copy link
Contributor Author

@Robbert zou jij hier het eindoordeel over willen vellen? #1784

@Robbert
Copy link
Member

Robbert commented Dec 15, 2024

Even gerebased om de preview te fixen

@rianrietveld
Copy link
Contributor

@Robbert @MarjonBakker Als dit alleen voor candidate is, kan ik ermee leven en laat ik het verder an Marjon over.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vaststellen Accessibility Supported Baseline
4 participants