I have a twig that displays a page sidebar twice on a page - once for mobile and next for the desktop. In such a case, the javascript on a facet displayed multiple times on a page does not work twice.

bug

CommentFileSizeAuthor
#6 same_facet_used_more-2910421-6.patch881 bytesb_sharpe
#3 Screen Shot 2017-09-21 at 13.38.20.png23.46 KBgauravjeet
#3 same_facet_used_more-2910421-3.patch733 bytesgauravjeet
Screen Shot 2017-09-21 at 13.37.21.png13.8 KBgauravjeet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gauravjeet created an issue. See original summary.

gauravjeet’s picture

gauravjeet CreditAttribution: gauravjeet as a volunteer and at Acro Commerce commented
Issue summary: View changes
gauravjeet’s picture

gauravjeet CreditAttribution: gauravjeet as a volunteer and at Acro Commerce commented
Status: Active » Needs review
FileSize
same_facet_used_more-2910421-3.patch733 bytes
Screen Shot 2017-09-21 at 13.38.20.png23.46 KB

Here is the patch. A screenshot of facets after applying this patch.

patch applied

gauravjeet’s picture

gauravjeet CreditAttribution: gauravjeet as a volunteer and at Acro Commerce commented
Title: Same facet used more than once on a page - soft limit does not work correctly » If same is facet used more than once on a page - soft limit does not work correctly
gauravjeet’s picture

gauravjeet CreditAttribution: gauravjeet as a volunteer and at Acro Commerce commented
Title: If same is facet used more than once on a page - soft limit does not work correctly » If same facet is used more than once on a page - soft limit does not work correctly
b_sharpe’s picture

b_sharpe CreditAttribution: b_sharpe at ImageX commented
Status: Needs review » Reviewed & tested by the community
FileSize
same_facet_used_more-2910421-6.patch881 bytes

Thanks, works great! Rerolled strictly so patch would apply properly in composer as had 'a/facets' path, no code change.

StryKaizer’s picture

StryKaizer
Primary language Dutch
Location Belgium
CreditAttribution: StryKaizer as a volunteer and at ClicketyClick commented
Status: Reviewed & tested by the community » Fixed

Thx all!

  • StryKaizer committed 3f08741 on 8.x-1.x authored by b_sharpe 
    Issue #2910421 by gauravjeet, b_sharpe: If same facet is used more than...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.