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]: <Accordion /> #2294

Closed

KenAJoh opened this issue Sep 18, 2023 · 5 comments · Fixed by #2310

Assignees

Labels

forespørsel 🥰 komponenter 🧩

Collaborator

KenAJoh commented Sep 18, 2023

Tilbakemelding

Ved hover på Accordion Header har innholdet ingen padding top

Screenshot 2023-09-18 at 09 10 08

// 0.5rem

// 0.25rem

KenAJoh added design 🎨 forespørsel 🥰 komponenter 🧩 labels

KenAJoh assigned sjur-gr

Contributor

sjur-gr commented Sep 18, 2023

Ser at vi har 0.75rem padding på høyre side, det samme på venstre uten indent. Tester med 0.75rem som padding top også. Funker det?
https://www.figma.com/proto/m0qpSPpsnNC9DwdmerGqJH/branch/onTL5HTNjRCMR2fsaVoAZS/Core-UI-kit?page-id=12133%3A32272&type=design&node-id=38524-3512&viewport=-1711%2C1161%2C0.57&t=CeAa43yUlk6M7guq-1&scaling=min-zoom&starting-point-node-id=38524%3A3512&mode=design

Collaborator Author

KenAJoh commented Sep 18, 2023

🙌 Fungerer det!

Contributor

sjur-gr commented Sep 18, 2023

Cool. Gi meg et hint før den går live, så skal jeg justere Figma samtidig.

sjur-gr assigned KenAJoh

KenAJoh mentioned this issue

Accordion padding og warnings #2310

Merged

Collaborator Author

KenAJoh commented Sep 24, 2023

Er klar til merge nå #2310

Contributor

sjur-gr commented Sep 26, 2023

Publisert i Figma

KenAJoh closed this as completed in #2310

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