search

nl-design-system / rijkshuisstijl-community

Unofficial Rijkshuisstijl components and design tokens based on the NL Design System architecture. This project is not endorsed by the Dutch Ministry of General Affairs.
https://rijkshuisstijl-community.vercel.app/
European Union Public License 1.2
6 stars 3 forks source link

Button component #455

Closed Rerbun closed 2 months ago

Rerbun commented 2 months ago

https://nl-design-system.github.io/utrecht/storybook-react/?path=/docs/react-button--docs

Figma Button component in NLDS - Rijksoverheid - Bibliotheek

sjimbonator commented 2 months ago

https://github.com/nl-design-system/rijkshuisstijl-community/blob/main/packages/components-react/src/Button.tsx -> word niet meer gebruikt dus kan weg

Rozerinay commented 1 month ago

Design review

De Icon only button heeft in Figma andere spacingwaarden dan de reguliere Button. Deze tokens hebben de prefix todo, wat betekent dat ze niet gebouwd zijn. Ik zie ook dat de Icon only button geen losse component is, maar een variant van de Utrecht-button met een aria-label.

Is het alsnog mogelijk om voor de Icon only variant toch aparte spacingwaarden toe te kennen, of niet @Rerbun?

Voor de rest ziet het er goed uit!

Rozerinay commented 1 month ago

Wijziging ziet er goed uit!