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]: <ExpansionCard /> #1869

Closed
Trent-NAV opened this issue Mar 21, 2023 · 3 comments
Closed

[Innspill til komponent]: <ExpansionCard /> #1869

Trent-NAV opened this issue Mar 21, 2023 · 3 comments
Assignees
@KenAJoh @olejorgenbakken
Labels
beta 🧪 forespørsel 🥰 Ønsker fra folk komponenter 🧩

Comments

@Trent-NAV
Copy link
Contributor

Trent-NAV commented Mar 21, 2023 •
edited
Loading

Tilbakemelding

Fint at det er laget en Small variant men ser at den kun reduserer paddingen noe, savner redusert bodytekst størrelse. Vi bruker ikke BodyLong/Medium til noe men har BodyLong/Small som standard, det er det mange saksbehandlingssystemer som har. Ser det er innebygget component variant på Title til Heading/Desktop/Small, så for å følge opp logikken der burde BodyLong/Small være en egen variant for bodytype eller description om den ikke inkluderes automatisk i Small varianten for hele komponentet.
@sjur-gr
Copy link
Contributor

sjur-gr commented Mar 21, 2023

Godt poeng. Det fikser vi.

@sjur-gr
Copy link
Contributor

sjur-gr commented Mar 21, 2023

Endringen er publisert i Figma

@Trent-NAV
Copy link
Contributor Author

Supert, takk 😀

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

@KenAJoh KenAJoh

@olejorgenbakken olejorgenbakken

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

No branches or pull requests
4 participants
@KenAJoh @olejorgenbakken @sjur-gr @Trent-NAV