Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Commit c7585eb

Browse files
author
Alexander Schmitz
committed
Rangeslider: added new event beforechange to slider which we return false on when we want to eat the change event in rangeslider
1 parent 89238c2 commit c7585eb

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

js/widgets/forms/rangeslider.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ define( [ "jquery", "../../jquery.mobile.core", "../../jquery.mobile.widget", ".
6161
"slidebeforestart": "_slidebeforestart",
6262
"slidestop": "_slidestop",
6363
"slidedrag": "_slidedrag",
64-
"change": "_change",
64+
"slidebeforechange": "_change",
6565
"blur": "_change",
6666
"keyup": "_change"
6767
});
@@ -156,11 +156,15 @@ define( [ "jquery", "../../jquery.mobile.core", "../../jquery.mobile.widget", ".
156156
if( min === max ) {
157157
thisSlider.data( "mobileSlider" ).handle.css( "z-index", 1 );
158158
otherSlider.data( "mobileSlider" ).handle.css( "z-index", 0 );
159+
159160
} else {
160161
otherSlider.data( "mobileSlider" ).handle.css( "z-index", "" );
161162
thisSlider.data( "mobileSlider" ).handle.css( "z-index", "" );
162163
}
163164
this._updateHighlight();
165+
if(min >= max){
166+
return false;
167+
}
164168
},
165169

166170
_updateHighlight: function() {

js/widgets/forms/slider.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -483,7 +483,11 @@ $.widget( "mobile.slider", $.mobile.widget, {
483483
control[ 0 ].selectedIndex = newval;
484484
}
485485
if ( !isfromControl && valueChanged ) {
486+
if ( this._trigger( "beforechange", event ) === false) {
487+
return false;
488+
}
486489
control.trigger( "change" );
490+
487491
}
488492
}
489493
},

0 commit comments

Comments
 (0)