Skip to content
Browse files

Slider: fixed step alignment to handle negative fractional values. Fi…

…xed #5583 - Slider displays negative fractional values incorrectly. Thanks for the patch watanabe.
  • Loading branch information...
1 parent b7c0823 commit 34912bc933d2787b0e3143b4dbd84e70bcc67928 @rdworth rdworth committed May 11, 2010
Showing with 3 additions and 3 deletions.
  1. +3 −3 ui/jquery.ui.slider.js
View
6 ui/jquery.ui.slider.js
@@ -591,12 +591,12 @@ $.widget( "ui.slider", $.ui.mouse, {
if ( val > this._valueMax() ) {
return this._valueMax();
}
- var step = this.options.step,
+ var step = ( this.options.step > 0 ) ? this.options.step : 1,
valModStep = val % step,
alignValue = val - valModStep;
- if ( valModStep >= ( step / 2 ) ) {
- alignValue += step;
+ if ( Math.abs(valModStep) * 2 >= step ) {
+ alignValue += ( valModStep > 0 ) ? step : ( -step );
}
// Since JavaScript has problems with large floats, round

0 comments on commit 34912bc

Please sign in to comment.
Something went wrong with that request. Please try again.