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

Action group #324

Open
5 of 13 tasks
AlineNap opened this issue Oct 31, 2024 · 1 comment
Open
5 of 13 tasks

Action group #324

AlineNap opened this issue Oct 31, 2024 · 1 comment
Labels
new component Adding a new component to the library react Issue involving a ReactJS component

Comments

@AlineNap
Copy link
Contributor

AlineNap commented Oct 31, 2024

Omschrijving

Groepeert één of meer gerelateerde acties en verzorgt de lay-out van de content.

Let op: Bij Utrecht heet het nog ButtonGroup, de NLDS-naam wordt ActionGroup, deze nemen wij alvast over.

Nieuwe feature: property justify met de waarden <undefined> (default); start, center, end, space-between.
Dit zet (lux) class-names (...--justify-start|center|end|space-between) die de justify-content CSS properties zetten.

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
Copy link
Contributor Author

AlineNap commented Nov 4, 2024

Stijling check vragen

  1. Kunnen we twee varianten hebben? 1 met een column-gap van 16px en 1 met een column-gap 'auto' a la full width zoals EVA heeft in haar formulieren ontwerp? Hoe los je dit op in de code? Een endum of boolean property erbij met full width ja of nee.
  2. De naam voor NL is action-group utrecht gebruikt button-group dat is Legacy. Willen en kunnen we daar iets mee?

@MrSkippy MrSkippy added new component Adding a new component to the library react Issue involving a ReactJS component labels Nov 7, 2024
@MrSkippy MrSkippy added the complete dod Complete the D.o.D. of the issue label Nov 19, 2024
@MrSkippy MrSkippy moved this from Backlog to Ready in Lux React Components Nov 19, 2024
@MrSkippy MrSkippy removed the complete dod Complete the D.o.D. of the issue label Nov 19, 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