Fixes #106 (rz-slider-on-change fires before value is changed)

Author: Valentin Hervieu

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "angularjs-slider",
-  "version": "0.1.25",
+  "version": "0.1.27",
   "homepage": "https://github.com/rzajac/angularjs-slider",
   "authors": [
     "Rafal Zajac <rzajac@gmail.com>",

demo/index.html

@@ -20,6 +20,7 @@ <h1>AngularJS Touch Slider</h1>
         <article>
             <h2>Min/max slider example</h2>
             Value: <pre>{{ slider_data | json }}</pre>
+            <p>Value linked on change: {{ otherData.value }}</p>
 
             <rzslider
                     rz-slider-floor="0.5"
@@ -131,9 +132,11 @@ <h2>Draggable range example</h2>
         };
 
         $scope.slider_data = {value: 1};
+        $scope.otherData = {value: 10};
 
         $scope.onChange = function() {
           console.info('changed', $scope.slider_data.value);
+          $scope.otherData.value = $scope.slider_data.value * 10;
         };
     });
 </script>

dist/rzslider.min.css

@@ -1,6 +1,6 @@
 {
   "name": "jusas-angularjs-slider",
-  "version": "0.1.25",
+  "version": "0.1.27",
   "description": "AngularJS slider directive with no external dependencies. Mobile friendly!.",
   "main": "rzslider.js",
   "repository": {

dist/rzslider.min.js

@@ -4,7 +4,7 @@
  * (c) Rafal Zajac <rzajac@gmail.com>
  * http://github.com/rzajac/angularjs-slider
  *
- * Version: v0.1.21
+ * Version: v0.1.27
  *
  * Licensed under the MIT license
  */
@@ -596,7 +596,10 @@ function throttle(func, wait, options) {
      */
     callOnChange: function() {
       if(this.scope.rzSliderOnChange) {
-        this.scope.rzSliderOnChange();
+        var self = this;
+        $timeout(function() {
+            self.scope.rzSliderOnChange();
+        });
       }
     },
 
@@ -608,7 +611,6 @@ function throttle(func, wait, options) {
      */
     updateHandles: function(which, newOffset)
     {
-      this.callOnChange();
       if(which === 'rzSliderModel')
       {
         this.updateLowHandle(newOffset);
@@ -1113,6 +1115,7 @@ function throttle(func, wait, options) {
       this.updateHandles('rzSliderModel', newMinOffset);
       this.updateHandles('rzSliderHigh', newMaxOffset);
       this.scope.$apply();
+      this.callOnChange();
     },
 
     /**
@@ -1135,6 +1138,7 @@ function throttle(func, wait, options) {
           this.maxH.addClass('rz-active');
            /* We need to apply here because we are not sure that we will enter the next block */
           this.scope.$apply();
+          this.callOnChange();
         }
         else if(this.tracking === 'rzSliderHigh' && newValue <= this.scope.rzSliderModel)
         {
@@ -1145,6 +1149,7 @@ function throttle(func, wait, options) {
           this.minH.addClass('rz-active');
            /* We need to apply here because we are not sure that we will enter the next block */
           this.scope.$apply();
+          this.callOnChange();
         }
       }
 
@@ -1153,6 +1158,7 @@ function throttle(func, wait, options) {
         this.scope[this.tracking] = newValue;
         this.updateHandles(this.tracking, newOffset);
         this.scope.$apply();
+        this.callOnChange();
       }
     },