I have a small feature request.
The module hides the fac-results by inserting a 'display:none' through javascript. This is a little bit problematic if you want to put some nice animation styling on it. Would it be possible to replace the inline styling with a class that hides the element?

You could make your own specific class, or re-use Drupal's '.hidden' class (which defaults to 'display:none').
That way, themers can easily override the CSS and create their own beautiful search results.

And if you're at it, the same could be done for the 'position: relative' inline style set on the form, so it can be easily overridden.

Comments

bskibinski created an issue. See original summary.

  • ea9c365 committed on 8.x-1.x
    Issue #2954608: Replace javascripts "display: none" with class
    
marty2081’s picture

Status: Active » Needs review

Good point. I have committed the suggested changes. Can you confirm they work for you?

marty2081’s picture

Status: Needs review » Fixed
bskibinski’s picture

Sorry, was a bit busy, but dubbel-checked it, and works like a charm! thanks a lot!

Status: Fixed » Closed (fixed)

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