Skip to content

Conversation

@zipper
Copy link
Owner

@zipper zipper commented May 2, 2025

  • 🐞 Media changes no longer affect unrelated collapsable items
    When the media condition changes for a group of collapsable items, the handleDefaultExpanded method now only processes items whose media matches the current condition.
  • 🐞 Filter expanded items by media query in getExpanded
    The getExpanded method now checks media query conditions, ensuring only items matching their media settings are considered expanded. Items without a matching media condition (collapsable is inactive) are ignored. This handles edge cases where items in the same group have mixed media conditions.
  • 🐞 Improved accordion behavior
    During accordion initialization, all items except the default expanded one are closed first, and only then is the default expanded item opened. Additionally, when collapsing other items upon expanding one, the logic avoids collapsing the item that will be re-expanded immediately afterward.

zipper added 3 commits May 2, 2025 12:58
When the media condition changes for a group of collapsable items, the `handleDefaultExpanded` method now only processes items whose media matches the current condition.
The `getExpanded` method now checks media query conditions, ensuring only items matching their media settings are considered expanded. Items without a matching media condition (collapsable is inactive) are ignored. This handles edge cases where items in the same group have mixed media conditions.
During accordion initialization, all items except the default expanded one are closed first, and only then is the default expanded item opened. Additionally, when collapsing other items upon expanding one, the logic avoids collapsing the item that will be re-expanded immediately afterward.
@zipper zipper merged commit 8e3a7c2 into master May 2, 2025
1 check passed
@zipper zipper deleted the responsive-and-accordion-fix branch May 2, 2025 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants