-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
Accordion component #11501
Accordion component #11501
Conversation
Size Change: 0 B Total Size: 1.02 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks nice! Was behaviour discussed in terms of "one opened at a time" vs "how many you want open"?
Good point, I knew there was something else to consider. I'll check with @kasiabbb |
TODO
|
For #11474
What does this change?
Adding a React component to display content in an accordion
accordion.mp4
The animation behaviour was deemed a nice-to-have and as such currently only works in Chrome and Edge because it relies on interpolate-size and ::details-content. Hopefully support will improve.
The default behaviour of the component is that all drawers can be opened independently (without the others closing), but there is an optional prop (
closeOthersOnOpen
) to override this behaviour.How to test
yarn cardigan
and view Accordion storyHow can we measure success?
We'll need this component for the exhibition access section
Have we considered potential risks?
n/a