Merge pull request #120 from AddSearch/sc-10289/fix-bug-in-calculate-…

…slider-ranges

[sc-10289] rounding the min-max value of input type=range

package.json

@@ -1,6 +1,6 @@
 {
   "name": "addsearch-search-ui",
-  "version": "0.8.9",
+  "version": "0.8.10",
   "description": "JavaScript library to develop Search UIs for the web",
   "main": "./dist/addsearch-search-ui.min.js",
   "jsdelivr": "./dist/addsearch-search-ui.min.js",

src/components/activefilters/activefilters.js

@@ -104,7 +104,13 @@ export default class ActiveFilters {
 
     // Range facets
     for (let field in filterState.activeRangeFacets) {
+      if (!filterState.activeRangeFacets.hasOwnProperty(field)) {
+        continue;
+      }
       for (let key in filterState.activeRangeFacets[field]) {
+        if (!filterState.activeRangeFacets[field].hasOwnProperty(key)) {
+          continue;
+        }
         active.push({
           name: field,
           type: TYPE.RANGE_FACET,

src/components/rangefacets/rangefacets.js

@@ -106,7 +106,9 @@ export default class RangeFacets {
             this.reduxStore.dispatch(setFieldStats(res.fieldStats, this.conf.field));
           });
         } else {
-          this.reduxStore.dispatch(setFieldStats(search.results.fieldStats, this.conf.field));
+          this.renderRangeSlider({
+            fieldStats: search.results.fieldStats
+          })
         }
 
       });

src/util/sliders.js

@@ -99,7 +99,9 @@ function UiRangeSlider() {
     sliderId = containerId;
 
     const [min, max, start, end] = getSliderValue();
-    initVisuals([min, max, start, end], settings.step);
+    const maxRounded = Math.ceil(parseFloat(max));
+    const minRounded = Math.floor(parseFloat(min));
+    initVisuals([minRounded, maxRounded, start, end], settings.step);
 
     const rangeStyles = settings.styles;
     const fromSlider = document.querySelector('#' + containerId + ' [data-id=adds-slider-control-from]');