From: poplix <poplix@papuasia.org>
Date: Mon, 6 Dec 2010 15:05:46 +0000 (-0500)
Subject: Slider: Changed _trimAlignValue function to return the correct inclusive value betwee... 
X-Git-Tag: 1.8.7~4
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=0d0969ca2b6ad5c8937313cc3868d8a21335d748;p=jquery-ui.git

Slider: Changed _trimAlignValue function to return the correct inclusive value between min and max. Fixes #6643 - using range and step options makes jquery change min and max.
---

diff --git a/ui/jquery.ui.slider.js b/ui/jquery.ui.slider.js
index ed27ae9c8..d3b4744e3 100644
--- 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 ) {