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

Single Checkbox #302

Open
1 of 13 tasks
AlineNap opened this issue Oct 14, 2024 · 3 comments
Open
1 of 13 tasks

Single Checkbox #302

AlineNap opened this issue Oct 14, 2024 · 3 comments
Assignees

Comments

@AlineNap
Copy link
Contributor

AlineNap commented Oct 14, 2024

Omschrijving

Dit wordt een nieuw component met prefix lux. Deze zullen wij mogelijk terugleveren aan NLDS.

Utrecht-checkbox kan gebruikt worden om dit component mee op te bouwen. Zie: #319

Definition of ready

  • Code check op community component gedaan door lux devs
  • Stijling check op community component gedaan door Aline
  • Plan voor missende features of bugs van het community component beschreven in het ticket
  • Component tokens zijn beschikbaar
  • Ticket doorgenomen met Vlad & Mark

Definition of done

Component

  • Component is beschikbaar in de index
  • Design tokens worden gebruikt

Storybook

  • Playground story
  • States en properties stories
  • Korte beschrijving van het component
    • Wat doet het
    • Basis van het component (bijv Utrecht)
    • Aanpassingen en/of uitbreidingen
    • Relatie met andere componenten (bijv. Form field label en Form field)

Tests

  • Het component rendert
  • Aanpassingen en/of uitbreidingen werken (zie ook Testing components)
  • Visual regression test (door LUX)

Links

@AlineNap
Copy link
Contributor Author

AlineNap commented Oct 14, 2024

Vragen uit stijlingcheck

Deze vragen beantwoorden in: #319 (comment)

  • Filled variant van utrecht-checkbox is nu nog niet mogelijk met design tokens op utrecht. Dit mogen we toevoegen op utrecht code. Jullie hebben eerder met Robbert gesproken over hoe dit component verbeterd kan worden. Graag issue maken op utrecht repo. Update: zie issue op utrecht repo, devs graag aanvullen.
  • Doen we tussenoplossing voor Mijn Aansluitingen? En wat is die oplossing dan?

@AlineNap AlineNap self-assigned this Oct 15, 2024
@AlineNap
Copy link
Contributor Author

AlineNap commented Oct 21, 2024

Wat is de reden dat dit een apart component is in code? Single checkbox? Kan iemand mij dat uitleggen? Dan kan ik er ook een discussie voor starten zodat de help wantend stappen doorlopen kunnen worden.
---> antwoord in comment hieronder.

Verder Vertelde Robbert mij dat hij dit pas ook heeft toegevoegd aan utrecht, dus wellicht kunnen we die hergebruiken?
En ik weet dat MO er ook 1 heeft, wellicht kunnen we die code gebruiken?

@AlineNap
Copy link
Contributor Author

AlineNap commented Oct 22, 2024

De single checkbox is niet de form-field-checkbox-option component omdat:

  • Andere appearance, stijling;
  • Mogelijkheid tot foutmelding op 1 checkbox (dit veld is verplicht);
    • --> Form-field component met een checkbox-group van 1 checkbox is geen optie omdat? Zijn er andere redenen dan geen label in het design?
  • Geen form-field-label in het design.
    • --> Wat als je het label verwijderd of hide in dit geval? Kan dat, heeft dat accessibility issues? Wat zijn redenen vanuit HTML/code om dat niet te doen?

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

No branches or pull requests

1 participant