-
Notifications
You must be signed in to change notification settings - Fork 19
Criteria voor Form Field Description #3587
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
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: daa4c60 The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
| tags: ["developer"], | ||
| }, | ||
| { | ||
| title: "Als je de tekst van de Form Field Description vergroot tot 200% blijft deze in zijn geheel zichtbaar.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Text-wrap toevoegen gebeurt niet in de component maar in de container, dus misschien moeten we deze SC opsplitsen en het text-wrap gedeeltte verplaatsen naar "bij gebruik"
| sc: "1.4.12", | ||
| status: "", | ||
| component: <Wcag1412 />, | ||
| tags: ["developer"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dit is te testen met een extensie zoals Stylus of User CSS
Dit staat in de 1.4.12 tekst in de uitklapper, maar bookmarklets werken niet goed in combinatie met een Content-Security-Policy. Het gaat te ver om dit kleine tekstje alle details uit te leggen, dus het is misschien het handigst dit tekstje te vervangen door een verwijzing naar "Hoe te testen" bij 1.4.12
| headingLevel="4" | ||
| items={[ | ||
| { | ||
| title: "De Form Field Description kan aan een input-element gekoppeld worden.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Misschien moeten we even bedenken of er een betere algemene term is die ook werkt voor andere elementen zoals textarea, select en custom elements zoals nl-combobox. Liefst gebaseerd op hoe de HTML of DOM standaarden het noemen
| import AcComponent from "@nl-design-system-unstable/documentation/componenten/ac/\_ac_component.md"; | ||
| import AcImplementatie from "@nl-design-system-unstable/documentation/componenten/ac/\_ac_implementatie.md"; | ||
| import IntroGebruik from "@nl-design-system-unstable/documentation/componenten/ac/\_component_gebruiken_intro.md"; | ||
| import Wcag131 from "@nl-design-system-unstable/documentation/componenten/ac/\_wcag-1.3.1-description.md"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zou het een idee zijn om voor algemene teksten de import Wcag131 te noemen, maar voor varianten die variant in de naam te gebruiken, zoals Wcag131FormFieldDescription? Dan kun je verderop goed zien welke algemene teksten hebben, en welke speciale teksten hebben.



notities:
preview: https://documentatie-jgwr1n8fe-nl-design-system.vercel.app/form-field-description/