- Issue created by @chrisolof
- Merge request !158Auto-expand collapsible checkboxes elements with one or more selections β (Merged) created by chrisolof
- First commit to issue fork.
- ππΊHungary mxr576 Hungary
I do not know how to provide test coverage for this without adding Facets 3 as a dependency, did not find a way to configure a similar behavior as "The facets exposed filter is configured to order any selected options above deselected options." which is the culprit of the issue.
- πΊπΈUnited States smustgrave
If itβs just dealing with checkboxes should be able to provide test coverage, we got some existing ones as is
- ππΊHungary mxr576 Hungary
@smustgrave tried that, added a new checkbox multivalue widget to bef_view but could not reproduce the issue, not even with sorting enabled on items. So the real culprit is the way how facets module reorders the active item to the top.
Since Facets 3 switched to BEF as dependency, maybe it becomes inevitable having a compatibility test with it since more and more issues popping up in BEF's issue queue from there. This is how I also got here this week :-)
- πΊπΈUnited States smustgrave
Not in front of my computer does facets happen to have a test we can steal?
- ππΊHungary mxr576 Hungary
Checked quickly and I have only found a basic test coverage in the related sub module which might no be even using BEF in testing atm based on the export test view config :see-no-evil:
https://git.drupalcode.org/project/facets/-/tree/3.0.x/modules/facets_ex...
- Status changed to Fixed
19 days ago 3:03pm 15 May 2025 - πΊπΈUnited States smustgrave
I'll merge this without test coverage as that may be a whole thing. But eventually probably should have a facets related test added
-
smustgrave β
committed 30d820e5 on 7.1.x
Issue #3512871 by mxr576, chrisolof: Automatic opening of collapsed...
-
smustgrave β
committed 30d820e5 on 7.1.x
- πΊπΈUnited States smustgrave
Eventually will release 7.1.x so merged to that too.
- ππΊHungary mxr576 Hungary
. But eventually probably should have a facets related test added
Agreed, thank you!
Automatically closed - issue fixed for 2 weeks with no activity.