Truly fix angular-slider#106. rz-slider-on-change fires before value …

…is changed
baelter · Aug 25, 2015 · 9358261 · 9358261
1 parent 4f7b9f5
commit 9358261
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 7 deletions.
diff --git a/bower.json b/bower.json
@@ -1,6 +1,6 @@
 {
   "name": "angularjs-slider",
-  "version": "0.1.26",
+  "version": "0.1.27",
   "homepage": "https://github.com/rzajac/angularjs-slider",
   "authors": [
     "Rafal Zajac <rzajac@gmail.com>",

diff --git a/demo/index.html b/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>

diff --git a/dist/rzslider.min.css b/dist/rzslider.min.css
diff --git a/dist/rzslider.min.js b/dist/rzslider.min.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "jusas-angularjs-slider",
-  "version": "0.1.26",
+  "version": "0.1.27",
   "description": "AngularJS slider directive with no external dependencies. Mobile friendly!.",
   "main": "rzslider.js",
   "repository": {

diff --git a/rzslider.js b/rzslider.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();
+        });
       }
     },