Slider: Changed _trimAlignValue function to return the correct inclus…

…ive value between min and max. Fixes #6643 - using range and step options makes jquery change min and max.
jquery · Dec 6, 2010 · 0d0969ca2b6ad5c8937313cc3868d8a21335d748 · 0d0969c
1 parent 22cf318
commit 0d0969ca2b6ad5c8937313cc3868d8a21335d748
Unified Split

Showing with 3 additions and 3 deletions.

+3 −3 ui/jquery.ui.slider.js
diff --git a/ui/jquery.ui.slider.js b/ui/jquery.ui.slider.js
@@ -585,14 +585,14 @@ $.widget( "ui.slider", $.ui.mouse, {
 
 	// returns the step-aligned value that val is closest to, between (inclusive) min and max
 	_trimAlignValue: function( val ) {
-		if ( val < this._valueMin() ) {
+		if ( val <= this._valueMin() ) {
 			return this._valueMin();
 		}
-		if ( val > this._valueMax() ) {
+		if ( val >= this._valueMax() ) {
 			return this._valueMax();
 		}
 		var step = ( this.options.step > 0 ) ? this.options.step : 1,
-			valModStep = val % step,
+			valModStep = (val - this._valueMin()) % step;
 			alignValue = val - valModStep;
 
 		if ( Math.abs(valModStep) * 2 >= step ) {