The code that's breaking it is in gesso_facetapi_link_inactive().

CommentFileSizeAuthor
#4 gesso-facets-that-use-checkbox-2551773-4.patch944 bytesmortona2k
#3 gesso-facets-that-use-checkbox-2551773-3.patch953 bytesmortona2k

Comments

mortona2k created an issue. See original summary.

mortona2k’s picture

mortona2k commented

The last line in gesso_facetapi_link_inactive replaces the facetapi-checkbox and facetapi-inactive classes with facet and is-inactive. However, the classes were used by facetapi.js, and elsewhere in facetapi and search_api_views.

I saw that this change was made in https://www.drupal.org/node/2452099.

mortona2k’s picture

mortona2k commented
StatusFileSize
new gesso-facets-that-use-checkbox-2551773-3.patch953 bytes

This patch adds classes to facet markup instead of replacing them

mortona2k’s picture

mortona2k commented
StatusFileSize
new gesso-facets-that-use-checkbox-2551773-4.patch944 bytes

  • dcmouyard committed 1ffa454 on 7.x-1.x authored by mortona2k 
    Issue #2551773 by mortona2k: Fixed classes for facetapi checkbox widget.
dcmouyard’s picture

dcmouyard commented
Status: Active » Fixed

Since these classes are needed for facetapi.js, I didn’t use the SMACSS classes theme setting to show/hide these classes.

Status: Fixed » Closed (fixed)

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