##fldc25

@brittany-willis, I've spent a lot of time fixing various issues with this module. For this reason before you evaluate anything with this module I highly recommend having a look at applying my merge request that fixes various bugs (likely more than advertised) as well as adds new functionality.

✨ Book TOC (Table of Contents) Active

@mherchel What do you think of the approach in:

https://developer.mozilla.org/en-US/blog/html-details-exclusive-accordions/

If not, I believe this is going to need JavaScript to work.

Thanks @vasantha deepika! For future reference, if you'd like someone else to review your work, please change the issue status to "Needs review" -- that'll ensure that other eyes get on it. :)

Thank you for the new design update! I have updated the MR based on the new design. Additionally, @mherchel, I have used the element for the accordion.

Wrong file! Actually updated now :)

Added the design to the IS, note that I've removed the CTA link because this won't work well in XB currently, with the restricted width that we have. So for simplicity let's leave it out for now.

Pushed up what I had to a new branch here: feature/1.x-accordion-component

@grifstuf was working on this at FLDC (see the first comment). I know he had code written, but not sure why it's not pushed.

Thanks for the feedback Jim Birch!
I’ll review the approach using the element and update the component accordingly.

Using the html element will negate the need for JavaScript to make the accordion work.

Here is a good blog about it.

https://developer.mozilla.org/en-US/blog/html-details-exclusive-accordions/

I assume there is some kind of component is that could be used for the unique name.

I have created the Accordion component, but a few improvements are still needed. I am actively working on them.
Please review it and share your suggestions, as they would be very helpful.

Thanks!

released with 1.0.9 →

vasantha deepika → made their first commit to this issue’s fork.

Should we call this something generic like Accordion

makes sense to me! Updating title