feat(enforceStep): Add an enforceStep option (defaults to true)

Author: Valentin Hervieu

README.md

@@ -184,6 +184,7 @@ The default options are:
     selectionBarColor: null,
     keyboardSupport: true,
     scale: 1,
+    enforceStep: true,
     enforceRange: false,
     noSwitching: false,
     onlyBindHandles: false,
@@ -258,6 +259,8 @@ $scope.slider = {
 
 **scale** - _Number (defaults to 1)_: If you display the slider in an element that uses `transform: scale(0.5)`, set the `scale` value to 2 so that the slider is rendered properly and the events are handled correctly.
 
+**enforceStep** - _Boolean (defaults to true)_: Set to true to force the value to be rounded to the step, even when modified from the outside.. When set to false, if the model values are modified from outside the slider, they are not rounded and can be between two steps.
+
 **enforceRange** - _Boolean (defaults to false)_: Set to true to round the `rzSliderModel` and `rzSliderHigh` to the slider range even when modified from outside the slider. When set to false, if the model values are modified from outside the slider, they are not rounded but they are still rendered properly on the slider.
 
 **noSwitching** - _Boolean (defaults to false)_: Set to true to prevent to user from switching the min and max handles. *Applies to range slider only.*

dist/rzslider.css

@@ -1,7 +1,7 @@
 /*! angularjs-slider - v2.6.0 - 
  (c) Rafal Zajac <rzajac@gmail.com>, Valentin Hervieu <valentin@hervieu.me>, Jussi Saarivirta <jusasi@gmail.com>, Angelin Sirbu <angelin.sirbu@gmail.com> - 
  https://github.com/angular-slider/angularjs-slider - 
- 2016-01-31 */
+ 2016-02-06 */
 rzslider {
   position: relative;
   display: inline-block;

dist/rzslider.js

@@ -1,7 +1,7 @@
 /*! angularjs-slider - v2.6.0 - 
  (c) Rafal Zajac <rzajac@gmail.com>, Valentin Hervieu <valentin@hervieu.me>, Jussi Saarivirta <jusasi@gmail.com>, Angelin Sirbu <angelin.sirbu@gmail.com> - 
  https://github.com/angular-slider/angularjs-slider - 
- 2016-02-03 */
+ 2016-02-06 */
 /*jslint unparam: true */
 /*global angular: false, console: false, define, module */
 (function(root, factory) {
@@ -52,6 +52,7 @@
       selectionBarColor: null,
       keyboardSupport: true,
       scale: 1,
+      enforceStep: true,
       enforceRange: false,
       noSwitching: false,
       onlyBindHandles: false,
@@ -636,9 +637,11 @@
 
         this.minValue = this.options.floor;
 
-        this.scope.rzSliderModel = this.roundStep(this.scope.rzSliderModel);
-        if (this.range)
-          this.scope.rzSliderHigh = this.roundStep(this.scope.rzSliderHigh);
+        if (this.options.enforceStep) {
+          this.scope.rzSliderModel = this.roundStep(this.scope.rzSliderModel);
+          if (this.range)
+            this.scope.rzSliderHigh = this.roundStep(this.scope.rzSliderHigh);
+        }
 
         if (this.options.ceil != null)
           this.maxValue = this.options.ceil;

dist/rzslider.min.css

@@ -56,6 +56,7 @@
       selectionBarColor: null,
       keyboardSupport: true,
       scale: 1,
+      enforceStep: true,
       enforceRange: false,
       noSwitching: false,
       onlyBindHandles: false,
@@ -640,9 +641,11 @@
 
         this.minValue = this.options.floor;
 
-        this.scope.rzSliderModel = this.roundStep(this.scope.rzSliderModel);
-        if (this.range)
-          this.scope.rzSliderHigh = this.roundStep(this.scope.rzSliderHigh);
+        if (this.options.enforceStep) {
+          this.scope.rzSliderModel = this.roundStep(this.scope.rzSliderModel);
+          if (this.range)
+            this.scope.rzSliderHigh = this.roundStep(this.scope.rzSliderHigh);
+        }
 
         if (this.options.ceil != null)
           this.maxValue = this.options.ceil;

dist/rzslider.min.js

@@ -104,14 +104,14 @@ describe('rzslider - ', function() {
       slider.resetSlider.called.should.be.true;
     });
 
-    it('should round the model value to the step', function() {
+    it('should round the model value to the step by default', function() {
       scope.slider.value = 54;
       scope.$digest();
       expect(scope.slider.value).to.equal(50);
 
       scope.slider.value = 55;
       scope.$digest();
-      $timeout.flush(); //to flush the throttle function
+      $timeout.flush(); //to flush the throttle function since we modify twice in a row
       expect(scope.slider.value).to.equal(60);
     });
 
@@ -317,6 +317,24 @@ describe('rzslider - ', function() {
         expect(slider.options.step).to.equal(1);
       });
 
+      it('should not round value to step if enforceStep is false', function() {
+        scope.slider.options.enforceStep = false;
+        scope.$digest();
+
+        scope.slider.value = 14;
+        scope.$digest();
+        expect(scope.slider.value).to.equal(14);
+      });
+
+      it('should round value to step if enforceStep is true', function() {
+        scope.slider.options.enforceStep = true;
+        scope.$digest();
+
+        scope.slider.value = 14;
+        scope.$digest();
+        expect(scope.slider.value).to.equal(10);
+      });
+
       it('should set the showTicks scope flag to true when showTicks is true', function() {
         scope.slider.options.showTicks = true;
         scope.$digest();