Range slider should use javascript events

Created on 1 June 2021, over 3 years ago
Updated 2 October 2023, about 1 year ago

The facets_range_widget doesn't trigger the 'facets_filtering' js event.

This is because it doesn't feature the class 'js-facets-widget' that is mandatory for this event to be triggered (see 'facets/js/base-widget.js'). It makes sense, since this is used also to trigger the 'window.location' change and the range widget has a logic of its own for redirecting.

Nevertheless, the range widget should trigger the 'facets_filtering' event on all facets. In my case, I am needing this in order to be able to disable the pointer events on any facet when the user has already submitted a facet.

My proposal is to make the range slider widget behave as the rest of widgets: instead of triggering the URL change by itself, use the small JS events API defined on base-widget.js.

Feature request
Status

Needs review

Version

3.0

Component

Code

Created by

🇪🇸Spain idiaz.roncero Madrid

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024