Slider: Proper animation of range

atomiomi · scottgonzalez · commit bf03479cc14a · 2015-04-06T16:59:45.000-04:00
Fixes #9459 Closes gh-1530
diff --git a/ui/slider.js b/ui/slider.js
@@ -610,13 +610,13 @@ return $.widget( "ui.slider", $.ui.mouse, {
 				this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { width: valPercent + "%" }, o.animate );
 			}
 			if ( oRange === "max" && this.orientation === "horizontal" ) {
-				this.range[ animate ? "animate" : "css" ]( { width: ( 100 - valPercent ) + "%" }, { queue: false, duration: o.animate } );
+				this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { width: ( 100 - valPercent ) + "%" }, o.animate );
 			}
 			if ( oRange === "min" && this.orientation === "vertical" ) {
 				this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { height: valPercent + "%" }, o.animate );
 			}
 			if ( oRange === "max" && this.orientation === "vertical" ) {
-				this.range[ animate ? "animate" : "css" ]( { height: ( 100 - valPercent ) + "%" }, { queue: false, duration: o.animate } );
+				this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { height: ( 100 - valPercent ) + "%" }, o.animate );
 			}
 		}
 	},

Original file line number	Diff line number	Diff line change
`@@ -610,13 +610,13 @@ return $.widget( "ui.slider", $.ui.mouse, {`
`610`	`610`	`this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { width: valPercent + "%" }, o.animate );`
`611`	`611`	`}`
`612`	`612`	`if ( oRange === "max" && this.orientation === "horizontal" ) {`
`613`		`- this.range[ animate ? "animate" : "css" ]( { width: ( 100 - valPercent ) + "%" }, { queue: false, duration: o.animate } );`
	`613`	`+ this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { width: ( 100 - valPercent ) + "%" }, o.animate );`
`614`	`614`	`}`
`615`	`615`	`if ( oRange === "min" && this.orientation === "vertical" ) {`
`616`	`616`	`this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { height: valPercent + "%" }, o.animate );`
`617`	`617`	`}`
`618`	`618`	`if ( oRange === "max" && this.orientation === "vertical" ) {`
`619`		`- this.range[ animate ? "animate" : "css" ]( { height: ( 100 - valPercent ) + "%" }, { queue: false, duration: o.animate } );`
	`619`	`+ this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { height: ( 100 - valPercent ) + "%" }, o.animate );`
`620`	`620`	`}`
`621`	`621`	`}`
`622`	`622`	`},`