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

Open
MeesD94 opened this issue Aug 21, 2024 · 6 comments
Open

Fieldset #569

MeesD94 opened this issue Aug 21, 2024 · 6 comments
Assignees

Comments

@MeesD94
Copy link
Contributor

MeesD94 commented Aug 21, 2024

No description provided.

@Rerbun Rerbun self-assigned this Aug 30, 2024
@Rerbun
Copy link
Contributor

Rerbun commented Sep 5, 2024

Eerste PR hiervoor is klaargezet (#609), maar er is geen styling in Figma

@Rerbun
Copy link
Contributor

Rerbun commented Sep 18, 2024

Fieldset section RVO

@Robbert
Copy link
Member

Robbert commented Sep 18, 2024

@Rozerinay Rozerinay assigned Rozerinay and AlineNap and unassigned Rozerinay Sep 23, 2024
@Rozerinay
Copy link
Contributor

Aline maakt hier tokens voor aan. Verder krijgt de Fieldset dezelfde tokens als de Form Field. Dit is een tussenoplossing.

@AlineNap
Copy link
Contributor

Design Review:

  • De stijling van Storybook en template komen niet overeen, zie discussie in thread. Daarvoor is deze PR gemaakt met design tokens voor de fieldset en heeft Ruben dit issue op de backlog gezet.
  • Zijn er afspraken gemaakt wat we wel en niet van de utrecht documentatie en code overnemen? Utrecht heeft bijvoorbeeld nog element, section en role in de playground. Ook geven ze nog extra uitleg etc. Ben benieuwd of dat bewust weggelaten/ anders is?

@AlineNap
Copy link
Contributor

Qua stijling is die nu goed. Vooral nog vragen over de documentatie van dit element: https://codefornl.slack.com/archives/C01DAT4TRPF/p1729505233953249?thread_ts=1729243165.047769&cid=C01DAT4TRPF.

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

No branches or pull requests

5 participants