Slider: Proper animation of range

atomiomi · scottgonzalez · commit a3bf7786d630 · 2016-06-09T13:24:36.000-04:00
Fixes #9459 Closes jquerygh-1530 (cherry picked from commit bf03479)
diff --git a/ui/slider.js b/ui/slider.js
@@ -625,13 +625,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
`@@ -625,13 +625,13 @@ return $.widget( "ui.slider", $.ui.mouse, {`
`625`	`625`	`this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { width: valPercent + "%" }, o.animate );`
`626`	`626`	`}`
`627`	`627`	`if ( oRange === "max" && this.orientation === "horizontal" ) {`
`628`		`- this.range[ animate ? "animate" : "css" ]( { width: ( 100 - valPercent ) + "%" }, { queue: false, duration: o.animate } );`
	`628`	`+ this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { width: ( 100 - valPercent ) + "%" }, o.animate );`
`629`	`629`	`}`
`630`	`630`	`if ( oRange === "min" && this.orientation === "vertical" ) {`
`631`	`631`	`this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { height: valPercent + "%" }, o.animate );`
`632`	`632`	`}`
`633`	`633`	`if ( oRange === "max" && this.orientation === "vertical" ) {`
`634`		`- this.range[ animate ? "animate" : "css" ]( { height: ( 100 - valPercent ) + "%" }, { queue: false, duration: o.animate } );`
	`634`	`+ this.range.stop( 1, 1 )[ animate ? "animate" : "css" ]( { height: ( 100 - valPercent ) + "%" }, o.animate );`
`635`	`635`	`}`
`636`	`636`	`}`
`637`	`637`	`},`