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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kevishie’s picture

kevishie CreditAttribution: kevishie commented
FileSize
facetapi-activelistitem-1482304.patch621 bytes

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

kevishie’s picture

kevishie CreditAttribution: kevishie commented
FileSize
facetapi-activelistitem-1482304-1.patch619 bytes

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

cpliakas’s picture

cpliakas CreditAttribution: cpliakas commented
Status: Active » Needs review

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

sachbearbeiter’s picture

sachbearbeiter CreditAttribution: sachbearbeiter commented

thanks a lot

cpliakas’s picture

cpliakas CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: cpliakas commented
Issue tags: +low hanging fruit

Adding to "low hanging fruit" list.

m1n0’s picture

m1n0 CreditAttribution: 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
CreditAttribution: fengtan commented
Status: Needs work » Needs review
FileSize
facetapi-activelistitem-1482304-9.patch2.01 KB

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

nimek’s picture

nimek CreditAttribution: nimek as a volunteer commented

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

adriancid’s picture

adriancid
Primary language Spanish
Location Montreal, Canada
CreditAttribution: adriancid as a volunteer 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 CreditAttribution: eugene.ilyin as a volunteer and at DrupalJedi commented

Thank you guys. Committed.

eugene.ilyin’s picture

eugene.ilyin CreditAttribution: eugene.ilyin as a volunteer and at DrupalJedi commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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