Show less is now working if block configuration links are rendered.
$(this).siblings() returns the configurations links.

CommentFileSizeAuthor
#2 facets-show-less-not-working-2801383-1.patch1009 bytesdragos-dumi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dragos.dumitrescu created an issue. See original summary.

dragos-dumi’s picture

dragos-dumi CreditAttribution: dragos-dumi as a volunteer commented
FileSize
facets-show-less-not-working-2801383-1.patch1009 bytes

Attached a patch to fix the selector

dragos-dumi’s picture

dragos-dumi CreditAttribution: dragos-dumi as a volunteer commented
Status: Active » Needs review
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and commented
Status: Needs review » Reviewed & tested by the community

I guess this makes sense. We need to fix #2794745: Use Search API's display plugin to fix facets before I'll commit any other changes.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and commented

This looks good, asking for another review tomorrow before committing.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and commented
Assigned: Unassigned » StryKaizer
StryKaizer’s picture

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

Committed, thanks!

Status: Fixed » Closed (fixed)

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