fix(slider): make modelValue be set on pressdown

Closes #1296.
angular · Jan 27, 2015 · 7028a75 · 7028a75
1 parent 8a0411f
commit 7028a75
Showing 1 changed file with 12 additions and 9 deletions.
diff --git a/src/components/slider/slider.js b/src/components/slider/slider.js
@@ -287,10 +287,21 @@ function SliderDirective($$rAF, $window, $mdAria, $mdUtil, $mdConstant, $mdThemi
       element[0].focus();
       refreshSliderDimensions();
 
-      setSliderFromEvent(ev);
+      var exactVal = percentToValue( positionToPercent( ev.pointer.x ));
+      var closestVal = minMaxValidator( stepValidator(exactVal) );
+      scope.$apply(function() {
+        setModelValue( closestVal );
+        setSliderPercent( valueToPercent(closestVal));
+      });
     }
     function onPressUp(ev) {
       element.removeClass('dragging active');
+
+      var exactVal = percentToValue( positionToPercent( ev.pointer.x ));
+      var closestVal = minMaxValidator( stepValidator(exactVal) );
+      scope.$apply(function() {
+        setModelValue(closestVal);
+      });
     }
     function onDragStart(ev) {
       if (isDisabledGetter(scope)) return;
@@ -309,14 +320,6 @@ function SliderDirective($$rAF, $window, $mdAria, $mdUtil, $mdConstant, $mdThemi
       if (!isDragging) return;
       ev.stopPropagation();
       isDragging = false;
-
-      var exactVal = percentToValue( positionToPercent( ev.pointer.x ));
-      var closestVal = minMaxValidator( stepValidator(exactVal) );
-
-      setSliderPercent( valueToPercent(closestVal));
-      $$rAF(function(){
-        setModelValue( closestVal );
-      });
     }
 
     function setSliderFromEvent(ev) {