From: Paul Bakaus <paul.bakaus@googlemail.com>
Date: Tue, 17 Feb 2009 11:13:30 +0000 (+0000)
Subject: slider: animate option was ignored if only one handle (fixes #4167 slider:animation... 
X-Git-Tag: 1.7~83
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=77043b9bf2baf00034463cf9dc690976e181b31c;p=jquery-ui.git

slider: animate option was ignored if only one handle (fixes #4167 slider:animation for single handles broken)
---

diff --git a/ui/ui.slider.js b/ui/ui.slider.js
index 75109b7f4..a9a9b4cdc 100644
--- a/ui/ui.slider.js
+++ b/ui/ui.slider.js
@@ -317,7 +317,7 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {
 				});
 				var otherVal = this.values(index ? 0 : 1);
 				if (allowed !== false) {
-					this.values(index, newVal, !( event.type == 'mousedown' && this.options.animate ));
+					this.values(index, newVal, ( event.type == 'mousedown' && this.options.animate ));
 				}
 			}
 
@@ -362,13 +362,11 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {
 
 	},
 
-	values: function(index, newValue, noAnimation) {
-
-		if(!this.options.animate) noAnimation = true;
+	values: function(index, newValue, animated) {
 
 		if (arguments.length > 1) {
 			this.options.values[index] = newValue;
-			this._refreshValue(!noAnimation);
+			this._refreshValue(animated);
 			this._change();
 		}
 
@@ -384,7 +382,7 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {
 
 	},
 
-	_setData: function(key, value) {
+	_setData: function(key, value, animated) {
 
 		$.widget.prototype._setData.apply(this, arguments);
 
@@ -396,10 +394,10 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, {
 				this.element
 					.removeClass("ui-slider-horizontal ui-slider-vertical")
 					.addClass("ui-slider-" + this.orientation);
-				this._refreshValue();
+				this._refreshValue(animated);
 				break;
 			case 'value':
-				this._refreshValue();
+				this._refreshValue(animated);
 				break;
 		}