Accordion: change: do animation similar to ExpansionCard #3491
Conversation
|
Storybook demo / Chromaticac61890cb | 91 components | 135 stories |
|
I did swap out inner padding with content margin (from child to parent whitespace), this made the animation work better for the box-shadow that was also moved. |
There was a problem hiding this comment.
Seems like this snippet breaks after these changes, needs to be updates so that the "last" border at the bottom matches rest of the accordions
.navds-accordion__item:last-child > .navds-accordion__content
There was a problem hiding this comment.
ah... right... i didn't try to open the last accordion element 😅
There was a problem hiding this comment.
I can't see a good way around adding another div here 😞, do you see one? (if you revert this to using padding instead of margins, then you'll see the animation looks wonky for the "line" on the left (box-shadow). but adding more divs is brutish, but works.
Co-authored-by: Ken <26967723+KenAJoh@users.noreply.github.com>
Description
https://github.com/navikt/team-aksel/issues/689
Component Checklist 📝
@navikt/core/css/config/_mappings.js)@navikt/core/css/tokens.json)@navikt/aksel-stylelint/src/deprecations.ts)@navikt/core/react/src/index.tsand@navikt/core/react/package.json)@navikt/core/css/index.css)<Component>: <gitmoji?> <Text>.E.g. "Button: ✨ Add feature xyz.")