Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Due to the merge of both of the following, the soft limit no longer works if multiple instances are on the page, this is due to 2910421 changing the ID of the facet when multiple instances exist.
Comment | File | Size | Author |
---|---|---|---|
#3 | softlimit.gif | 219.86 KB | borisson_ |
#2 | multiple-instances-softlimit-2927283-2.patch | 1.4 KB | b_sharpe |
Comments
Comment #2
b_sharpe CreditAttribution: b_sharpe at ImageX commentedThis patch cleans up the code so that it has a known facet id to use throughout the function, though I would think this should be refactored in general to NOT change the id and maybe create a new data attribute for multiple instances as I can imagine this conflict may occur elsewhere.
Comment #3
borisson_I can't reproduce this.
Looks like this just works.
Comment #4
b_sharpe CreditAttribution: b_sharpe at ImageX commentedIt's referring to multiple instances of the SAME facet on the page (in my case due to mobile layout being radically different).
See #2910421: If same facet is used more than once on a page - soft limit does not work correctly for more info
Comment #5
borisson_Aha, thanks for explaining further. I tested the patch and it looks solid. Great work!
Comment #6
dcam CreditAttribution: dcam commentedRTBC +1
I just ran into this issue too because of a duplicate block in an off-canvas menu. The patch resolved the problem.
Comment #8
borisson_