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.

Sign up for GitHub

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

Jump to bottom

[Innspill til komponent]: <ErrorSummary /> #2244

Closed
JonathanVidmar opened this issue Sep 5, 2023 · 2 comments · Fixed by #2256
Closed

[Innspill til komponent]: <ErrorSummary /> #2244

JonathanVidmar opened this issue Sep 5, 2023 · 2 comments · Fixed by #2256
Assignees
@KenAJoh
Labels
forespørsel 🥰 Ønsker fra folk komponenter 🧩

Comments

@JonathanVidmar
Copy link

Tilbakemelding

Legge til border radius som en token man kan overstyre.
@KenAJoh
Copy link
Collaborator

KenAJoh commented Sep 6, 2023 •
edited
Loading

Hei 👋 Har dere en spesifikk border-radius dere tenkte å bruke? Vi vurderer å endre standard border-radius til 8px 🤔 Slipper da en egen token for det. 🤞

Screenshot 2023-09-06 at 11 14 14

@JonathanVidmar
Copy link
Author

Vi tänkte 8px på denna faktiskt 😅 Så på denna komponenten ville inte en token behövts i så fall 👌

@KenAJoh KenAJoh mentioned this issue Sep 7, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KenAJoh KenAJoh

Labels
forespørsel 🥰 Ønsker fra folk komponenter 🧩
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

🎨 ErrorSummary border-radius navikt/aksel
2 participants
@JonathanVidmar @KenAJoh