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

🎨👀 Design Review: Form Field radio group #398

Open
3 tasks
AlineNap opened this issue Feb 11, 2025 · 0 comments
Open
3 tasks

🎨👀 Design Review: Form Field radio group #398

AlineNap opened this issue Feb 11, 2025 · 0 comments

Comments

@AlineNap
Copy link
Contributor

AlineNap commented Feb 11, 2025

Feedback op basis van de visual regression:

  • Form field radio group invalid mist een streep aan de voorkant, deze graag toevoegen zodat die consistent is met de andere Form Fields
  • Bij radio group disabled zou de hele group en het label disabled moeten zijn.
  • Error message positie: Bij de Radio group staat de foutmelding boven het invoerveld, terwijl deze bij andere form fields onder de input staat. De radio group heeft de correct weergave, maar dit moet consistent worden toegepast.

Openstaande vraag

📌 Willen we dit component nog aanbieden als community component?
Wat was onze oorspronkelijke reden om deze zelf te bouwen? Dit kan invloed hebben op onderhoud en adoptie binnen de NLDS-community.

Acceptatiecriteria

  • De visuele weergave van Radio group (invalid) is correct en bevat de ontbrekende streep.
  • Radio group (disabled) is volledig uitgeschakeld, inclusief het label.
  • Beslissing over het aanbieden als community component is besproken en gedocumenteerd.
@AlineNap AlineNap converted this from a draft issue Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: To do
Development

No branches or pull requests

1 participant