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

Fieldset #298

Open
5 of 13 tasks
AlineNap opened this issue Oct 14, 2024 · 2 comments
Open
5 of 13 tasks

Fieldset #298

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

Comments

@AlineNap
Copy link
Contributor

AlineNap commented Oct 14, 2024

Omschrijving

Groep van gerelateerde formuliervelden.

Het component bestaat niet in de Utrecht React Componenten, dus dit wordt een eigen Lux component, wel met de css styling en class-names van Utrecht.
Voor de div-variant kan direct ook de role e.d. er goed opgezet worden. Zie de beschrijving bij Utrecht CSS Storybook. Een voorbeeld met de goede labelling is ook onderdeel van de DoD.

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

Links

@AlineNap AlineNap converted this from a draft issue Oct 14, 2024
@AlineNap AlineNap changed the title Form Fieldset Fieldset Oct 14, 2024
@AlineNap
Copy link
Contributor Author

AlineNap commented Oct 14, 2024

Vragen uit stijlingcheck:

Deze zijn besproken en beantwoord bij refinement van 15 oktober met Remy:

  • Hoe stijlen we de description voor Eenvoudig aansluiten? Ik heb nu de paragraph tokens toegepast, werkt dat?
    • Yes dat is de weg om in te slaan dus prefix.form-field.description.* tokens zijn niet nodig.
  • Hoe stijlen we de description idealiter, prefix.form-fieldset.description.* tokens nodig? PR op Utrecht?
    • Same als vraag 1, is vergelijkbaar met de paragraph in de alert.
  • Token nodig voor spacing tussen form elementen?
    • yes, eerst met prefix lux en parallel aanvragen bij utrecht?
  • Token nodig voor spacing tussen legend en description?
    • yes, eerst met prefix lux en parallel aanvragen bij utrecht? Gaan nog discussie aan/ uitzoeken hoe teksten te groeperen, zie slack thread.

Op 21 oktober tokens toegevoegd voor invalid, deze verwezen naar utrecht form-field invalid tokens. Als oplossing voor onderstaande vraag:

  • Wat doe ik met de invalid tokens? Ga er vanuit dat die alleen gebruikt worden voor radio en checkbox en dat dit soort geneste tokens zijn, dus dat ik ze niet hoef in te vullen. Klopt dat? PS. Robbert en jullie hadden ideeën voor verbeteren radio en checkbox groep, is denk relevant hiervoor. Zijn deze tokens dan wel nodig?

Deze nog uitzoeken:

  • Wat was tussenoplossing van Robbert voor missen van description? Hij had iets over een area-label toevoegen, jullie zeide dit te begrijpen woensdag. Voeg dit toe aan dit ticket zodat de devs weten wat ze moeten doen.
  • Waarvoor wordt de section tokens gebruikt? --> Aan Utrecht vragen.
  • Disabled willen wij niet toelaten bij LUX, moet ik dan die token wel invullen? En kunnen jullie die variant hiden/ niet beschikbaar zetten.

@AlineNap AlineNap mentioned this issue Oct 14, 2024
3 tasks
@AlineNap
Copy link
Contributor Author

Refinement 22 oktober:

AlineNap added a commit that referenced this issue Oct 23, 2024
# Contents

Nieuwe tokens: utrecht component fieldset

## 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

Relevant voor ticket: #298

---------

Co-authored-by: Remy Parzinski <[email protected]>
@MrSkippy MrSkippy added new component Adding a new component to the library react Issue involving a ReactJS component labels Nov 4, 2024
@MrSkippy MrSkippy moved this from Backlog to Ready in Lux React Components Nov 5, 2024
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: Ready
Development

No branches or pull requests

2 participants