Skip to content

Commit 354682e

Browse files
committed
Slider: added support for canceling start event. Fixes #5472 - slider: event start not cancelable
1 parent 5ffd3ab commit 354682e

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

ui/jquery.ui.slider.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,10 @@ $.widget("ui.slider", $.ui.mouse, {
147147
if (!self._keySliding) {
148148
self._keySliding = true;
149149
$(this).addClass("ui-state-active");
150-
self._start(event, index);
150+
var allowed = self._start(event, index);
151+
if (allowed === false) {
152+
return;
153+
}
151154
}
152155
break;
153156
}
@@ -262,7 +265,10 @@ $.widget("ui.slider", $.ui.mouse, {
262265
closestHandle = $(this.handles[++index]);
263266
}
264267

265-
this._start(event, index);
268+
var allowed = this._start(event, index);
269+
if (allowed === false) {
270+
return false;
271+
}
266272
this._mouseSliding = true;
267273

268274
self._handleIndex = index;
@@ -354,7 +360,7 @@ $.widget("ui.slider", $.ui.mouse, {
354360
uiHash.value = this.values(index);
355361
uiHash.values = this.values();
356362
}
357-
this._trigger("start", event, uiHash);
363+
return this._trigger("start", event, uiHash);
358364
},
359365

360366
_slide: function(event, index, newVal) {

0 commit comments

Comments
 (0)