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

Checkbox #319

Open
11 of 13 tasks
AlineNap opened this issue Oct 23, 2024 · 2 comments
Open
11 of 13 tasks

Checkbox #319

AlineNap opened this issue Oct 23, 2024 · 2 comments
Assignees
Labels
new component Adding a new component to the library react Issue involving a ReactJS component

Comments

@AlineNap
Copy link
Contributor

AlineNap commented Oct 23, 2024

Omschrijving

Invoerveld voor het kiezen van een optie uit een groep van opties, of om te kiezen tussen "wel" of "niet".

Let op: Dit alleen het input-element, om het een label, description en foutmelding te geven, moet het samengesteld worden met een FormField en FormField* componenten

Links

Checks

Definition of ready

  • Code check op community component (door LUX)
  • Styling check op community component (door LUX)
  • Plan voor missende features of bugs community component beschreven in ticket
  • Component tokens zijn beschikbaar op main
  • Ticket doorgenomen met uitvoerder

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
    • Wat doen (speciale) features
    • 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)
@AlineNap AlineNap converted this from a draft issue Oct 23, 2024
@AlineNap
Copy link
Contributor Author

Styling check

@VladAfanasev VladAfanasev self-assigned this Nov 4, 2024
@AlineNap AlineNap mentioned this issue Nov 6, 2024
3 tasks
MrSkippy pushed a commit that referenced this issue Nov 6, 2024
# Contents

- Nieuwe tokens: utrecht component checkbox

Ticket: #319

## Checklist

<!-- Surround an item with double tildes `~~` to indicate that it does
not apply to this PR -->

- [ ] New features/components and bugfixes are covered by tests
- [x] Changesets are created
- [ ] Definition of Done is checked

---------

Co-authored-by: Jaap-Hein Wester <[email protected]>
@MrSkippy MrSkippy added new component Adding a new component to the library react Issue involving a ReactJS component labels Nov 7, 2024
@AlineNap
Copy link
Contributor Author

Design review 18 December

  1. Waar komt de variant with target vandaan? Te vinden helemaal onderin Storybook, zie dit niet terug bij Utrecht doc. Verder wel nice, maar hoe werkt dit? Kan de achtergrond worden gestijlt? Wanneer is de target zichtbaar? etc.
  2. De checkbox is 24px bij 24px, zou 18px verwachten omdat de token utrecht.checkbox.size daarop is ingesteld. Hoe komt dit?
  3. Om de checkbox staat een div utrecht document die 28px hoog is. Waarom is die 2px hoger dan de checkbox zelf? Levert dat geen problemen op als het component wordt gebruikt in context?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new component Adding a new component to the library react Issue involving a ReactJS component
Projects
Status: In design review
Development

No branches or pull requests

3 participants