Closed callinmullaney closed 1 year ago
@callinmullaney I also pushed up a fix for a bug I noticed. If you 'click' on the icon in the 'Expand All' button it does nothing because the event target is the icon and not the button, which is what the JS is looking for.
@ryanhagerty Thanks for the solid review as always. I think I addressed all your comments here and should be ready for a final review.
:tada: This PR is included in version 1.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:
Summary
Explain the motivation for making this change. What existing problem does the pull request solve?
Documentation Update (required)
How to review this PR
npm install
thennpm run develop
Molecules/Accordion
and confirm the expand/collapse functionality for individual items and global controls works as expectedNotes: