i have to theme the selected facet item - but there is no active class at the list item ...

<li class="leaf first">
  <a class="facetapi-active" rel="nofollow" href="/filter">(-)</a>
Internet Service Providers
</li>

this would be wonderful ....

CommentFileSizeAuthor
#9 facetapi-activelistitem-1482304-9.patch2.01 KBfengtan
#2 facetapi-activelistitem-1482304-1.patch619 byteskevishie
#1 facetapi-activelistitem-1482304.patch621 byteskevishie

Comments

kevishie’s picture

kevishie commented
StatusFileSize
new facetapi-activelistitem-1482304.patch621 bytes

I had the same problem and this worked for me. Here is the patch..

kevishie’s picture

kevishie commented
StatusFileSize
new facetapi-activelistitem-1482304-1.patch619 bytes

That patch gave an error about whitespace...sorry...here is the latest..

cpliakas’s picture

cpliakas commented
Status: Active » Needs review

Thanks for the patch! Marking as needs review...

sachbearbeiter’s picture

sachbearbeiter commented

thanks a lot

cpliakas’s picture

cpliakas commented
Status: Needs review » Needs work

This looks great! Marking as needs work, only because it would be great to expand this to the Current Search blocks as well. The methods where similar code needs to be added are listed below:

Thanks,
Chris

cpliakas’s picture

cpliakas commented
Version: 7.x-2.x-dev » 7.x-1.x-dev

Marking as a 1.x feature. Although we have a 2.x branch, it is exactly the same at this point.

cpliakas’s picture

cpliakas commented
Issue tags: +low hanging fruit

Adding to "low hanging fruit" list.

m1n0’s picture

m1n0 commented
Issue summary: View changes

Why do we need 'active' class for Current Search block? It displays only active items already, so it might be unnecessary markup.

fengtan’s picture

fengtan
Location Montreal, Canada
commented
Status: Needs work » Needs review
StatusFileSize
new facetapi-activelistitem-1482304-9.patch2.01 KB

Here is an attempt to expand the feature to the Current Search blocks.

nimek’s picture

nimek commented

Patch from 9 works great +1 for commit very useful feauture

adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Status: Needs review » Reviewed & tested by the community

The last submitted patch, 1: facetapi-activelistitem-1482304.patch, failed testing.

The last submitted patch, 2: facetapi-activelistitem-1482304-1.patch, failed testing.

  • eugene.ilyin committed e3be721 on 7.x-1.x authored by fengtan 
    Issue #1482304 by kevishie, fengtan: Add class for active list item
eugene.ilyin’s picture

eugene.ilyin commented

Thank you guys. Committed.

eugene.ilyin’s picture

eugene.ilyin commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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