Ajax Facets do not support Range Slider

Patch which adds Ajax Facets support.

Thanks for the review!

The issue is that the FacetBlockAjaxController disregards all JS settings by using `renderPlain` rather than `render`. I'm not sure why it doesnt just use render.

In my case it seems to work by just switching it. Unfortuantely I hit another issue where my Views DOM ID didn't match so I applied https://www.drupal.org/project/facets/issues/3013795 which fixed. My own patch now depends on that one.

Another issue I ran into was that the facet filtering is lost when you filter using Views Exposed Filters which I also fixed for my own use case.

I'm currently running low on time so I dont have time to split these fixes across all the different issues therefore this patch is not mergable at the moment (as it depends on another patch). Once I have some spare time I can look into it again but if someone else works on it this should be a good start.

Patch(es) worked for us, thanks oxyc.

@oxyc
Hi
I'm sorry, but #7 patch is not applicable anymore...

Looks like this code is quite parallel to other issues. It would be good to consolidate and join forces.

This patch allows to solve the problem of an ajax facet in a search page and does not include a facet range widget, we are not obliged to apply an additional patch, it is based on the patch https:// www.drupal.org/files/issues/2018-11-16/ajax_facets_range_widget-3012531-...

I tried to re-roll the patch #7 against the latest dev release and the range facets still not use ajax.
#12 instead, worked for me and solved the problem (now the range slider facets use ajax).

Applying the patch on https://www.drupal.org/project/facets/issues/3052574 has no effect on the range widgets with ajax.

The patch #12 makes sliders work with ajax, but is reseting other filters. Here is a patch fixing it. Basically it gets the current request and merge the query params within the new proposed url. I now the code is ugly and there is a mostly duplicated loop but it works as starting point and maybe it can be improved by someone else.

The previous patch appends again and again the slider parameter to the url instead of replacing the existing one. Here is a improved version solving this issue.

The previous patch was wrong, this is the good one.

Is there a way to trigger range sldier facet with AJAX from other places like this?

$(this).trigger('facets_filter', [ $(this).val() ]);

@yivanov The event name is "slidestop". See https://api.jqueryui.com/slider/#event-stop

Hi, @akalam, I managed to do it like this

            $(facet).slider('values', [min,max]);
            $(facet).slider('option', 'stop')(null, { values: [min,max]});

The patch #16 is working almost perfectly but in my case I have 2 Range Slider facets on the same page. When I update the first one, the second is not updated.

Any fix for this issue ?

I have also the issue with other facets type. If I change one facets, the Range Slider facets are not updated

UPDATE :

So after some test, I applied the patch in #16 + replace all renderPlain by render in /src/Controller/FacetBlockAjaxController.php

And It's working now.

The patch in #22

Thank's to akalam and oxyc

I've faced with js error when using a non-range slider and value provided instead of values.
Created 2 patches, one from #16 and one from #21, because I have the fix #21 from patch #3052574: Facets with AJAX not working in most of situations.

Sorry, the previously provided patches applied incorrectly in js, corrected.

Patch 24 doesn't work. I have this js error:

Uncaught TypeError: settings.facets.sliders[facetId].urls is undefined

Patch 21 works.

Added fix for error from comment #25.

Now it works.
thank

Thank you so much all of you for these wonderful patches !
Unfortunately, patch from #26 does not work for facets with source using a specific "filter_key" (different than the hardcoded "f" value in the patch). Here is a new one supporting this feature.

Please note that the patch only works flawlessly on my side using the patch with the render fix.

Patch #28 works.
thank

I merged the work from #28 (with render fix) into the work at #3052574: Facets with AJAX not working in most of situations because the two conflict with each other. Should the work on this issue continue over there?

#31 works for me. Thank you.

fix #28 range slider not support multi field in same time

#43 have missing some code.

fix #28 range slider not support multi field in same time.
this patch base on #28

In order to be considered accessible, range sliders need part of a form with a label: https://www.a11ywithlindsey.com/blog/creating-accessible-range-slider-css

Rerolled patch from #28 for version 2.0.6.
I did not include any changes from #33, #34, #35 as it seemed to be related to some other bug and not relevant here.
Thank you.

This works great to make the slider ajaxify.

Sorry for the noise, I was testing patches and found out that #3013795: Robustify how ajax finds the facet block is already fixing this one, so I would suggest to deprecate this patch in favor of the linked one, as it covers more ground.

Updated patch. When you are applying a range slider filter without any other filter its add a base URL to filter values