Currently you can theme the active links of a current search block by overriding theme_facetapi_link_active, however the search keys are not themeable, they just run through check_plain().

It would be nice if they were themeable.

Then it would be possible to do something like run it through theme('facetapi_link_active') so it behaves like other facets.

CommentFileSizeAuthor
#3 themable-keys-1514304-3.patch2.16 KBcpliakas
#1 facetapi-themeable_keys_facet-1514304-1.patch2.09 KBrooby

Comments

rooby’s picture

rooby commented
Status: Active » Needs review
StatusFileSize
new facetapi-themeable_keys_facet-1514304-1.patch2.09 KB

Here is a patch to add this.

cpliakas’s picture

cpliakas commented

This makes sense to me! I'll test it in the next couple of days unless someone wants to beat me to it (which would be great).

Thanks,
Chris

cpliakas’s picture

cpliakas commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new themable-keys-1514304-3.patch2.16 KB

Patch works as advertised. Re-rolling with a minor code style change to eliminate two calls to getSearchKeys(), and also changing theme function from "current_search_search_keys" to just "current_search_keys". It seems inline with other theme functions such as "current_search_text", and I always opt for shorter names :-).

Thanks for the contribution, and sorry this slipped my radar.
Chris

cpliakas’s picture

cpliakas commented
Status: Reviewed & tested by the community » Fixed

Committed to all active versions of Facet API.

rooby’s picture

rooby commented

Great, thank you.

Status: Fixed » Closed (fixed)

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