noUiSlider 14.0.0

leongersen · Jun 20, 2019 · 19d0254 · 19d0254 · bjarnef · Jun 21, 2019
1 parent 809eb82
commit 19d0254
Show file tree

Hide file tree

Showing 6 changed files with 45 additions and 18 deletions.
diff --git a/CHANGELOG.MD b/CHANGELOG.MD
@@ -1,5 +1,11 @@
 # Changelog
 
+### 14.0.0 (*2019-06-20*)
+- Fixed: `change` & `slide` events should fire on keyboard control (#994);
+- Fixed: `.noUi-origin` overflows document on vertical sliders (#987);
+- Fixed: Clicking to right of handle doesn't move it when it's at the same point as another (#965);
+- Added: Additional documentation on number formatting (#978, #985);
+
 ### 13.1.5 (*2019-04-24*)
 - Fixed: Full-range padding (#880);
 

diff --git a/distribute/nouislider.css b/distribute/nouislider.css
@@ -1,4 +1,4 @@
-/*! nouislider - 13.1.5 - 4/24/2019 */
+/*! nouislider - 14.0.0 - 6/20/2019 */
 /* Functional styling;
  * These styles are required for noUiSlider to function.
  * You don't need to change these rules to apply your design.
@@ -40,14 +40,20 @@
   z-index: 1;
   top: 0;
   left: 0;
-  height: 100%;
-  width: 100%;
   -ms-transform-origin: 0 0;
   -webkit-transform-origin: 0 0;
   -webkit-transform-style: preserve-3d;
   transform-origin: 0 0;
   transform-style: flat;
 }
+.noUi-connect {
+  height: 100%;
+  width: 100%;
+}
+.noUi-origin {
+  height: 1%;
+  width: 1%;
+}
 /* Offset direction
  */
 html:not([dir="rtl"]) .noUi-horizontal .noUi-origin {

diff --git a/distribute/nouislider.js b/distribute/nouislider.js
@@ -1,4 +1,4 @@
-/*! nouislider - 13.1.5 - 4/24/2019 */
+/*! nouislider - 14.0.0 - 6/20/2019 */
 (function(factory) {
     if (typeof define === "function" && define.amd) {
         // AMD. Register as an anonymous module.
@@ -13,7 +13,7 @@
 })(function() {
     "use strict";
 
-    var VERSION = "13.1.5";
+    var VERSION = "14.0.0";
 
     //region Helper Methods
 
@@ -1567,8 +1567,8 @@
         }
 
         // Find handle closest to a certain percentage on the slider
-        function getClosestHandle(proposal) {
-            var closest = 100;
+        function getClosestHandle(clickedPosition) {
+            var smallestDifference = 100;
             var handleNumber = false;
 
             scope_Handles.forEach(function(handle, index) {
@@ -1577,11 +1577,19 @@
                     return;
                 }
 
-                var pos = Math.abs(scope_Locations[index] - proposal);
+                var handlePosition = scope_Locations[index];
+                var differenceWithThisHandle = Math.abs(handlePosition - clickedPosition);
 
-                if (pos < closest || (pos === 100 && closest === 100)) {
+                // Initial state
+                var clickAtEdge = differenceWithThisHandle === 100 && smallestDifference === 100;
+
+                // Difference with this handle is smaller than the previously checked handle
+                var isCloser = differenceWithThisHandle < smallestDifference;
+                var isCloserAfter = differenceWithThisHandle <= smallestDifference && clickedPosition > handlePosition;
+
+                if (isCloser || isCloserAfter || clickAtEdge) {
                     handleNumber = index;
-                    closest = pos;
+                    smallestDifference = differenceWithThisHandle;
                 }
             });
 
@@ -1830,7 +1838,12 @@
             // Decrement for down steps
             step = (isDown ? -1 : 1) * step;
 
-            valueSetHandle(handleNumber, scope_Values[handleNumber] + step, true);
+            setHandle(handleNumber, resolveToValue(scope_Values[handleNumber] + step, handleNumber), true, true);
+
+            fireEvent("slide", handleNumber);
+            fireEvent("update", handleNumber);
+            fireEvent("change", handleNumber);
+            fireEvent("set", handleNumber);
 
             return false;
         }
@@ -2080,8 +2093,10 @@
             // Convert the value to the slider stepping/range.
             scope_Values[handleNumber] = scope_Spectrum.fromStepping(to);
 
-            var rule = "translate(" + inRuleOrder(transformDirection(to, 0) - scope_DirOffset + "%", "0") + ")";
-            scope_Handles[handleNumber].style[options.transformRule] = rule;
+            var translation = 100 * (transformDirection(to, 0) - scope_DirOffset);
+            var translateRule = "translate(" + inRuleOrder(translation + "%", "0") + ")";
+
+            scope_Handles[handleNumber].style[options.transformRule] = translateRule;
 
             updateConnect(handleNumber);
             updateConnect(handleNumber + 1);

diff --git a/distribute/nouislider.min.css b/distribute/nouislider.min.css
diff --git a/distribute/nouislider.min.js b/distribute/nouislider.min.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "nouislider",
-  "version": "13.1.5",
+  "version": "14.0.0",
   "main": "distribute/nouislider.js",
   "style": "distribute/nouislider.min.css",
   "license": "MIT",