Slider: added support for canceling start event. Fixes #5472 - slider…

…: event start not cancelable
jquery · Apr 1, 2010 · 354682e · 354682e
1 parent 5ffd3ab
commit 354682e
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/ui/jquery.ui.slider.js b/ui/jquery.ui.slider.js
@@ -147,7 +147,10 @@ $.widget("ui.slider", $.ui.mouse, {
 					if (!self._keySliding) {
 						self._keySliding = true;
 						$(this).addClass("ui-state-active");
-						self._start(event, index);
+						var allowed = self._start(event, index);
+						if (allowed === false) {
+							return;
+						}
 					}
 					break;
 			}
@@ -262,7 +265,10 @@ $.widget("ui.slider", $.ui.mouse, {
 			closestHandle = $(this.handles[++index]);
 		}
 
-		this._start(event, index);
+		var allowed = this._start(event, index);
+		if (allowed === false) {
+			return false;
+		}
 		this._mouseSliding = true;
 
 		self._handleIndex = index;
@@ -354,7 +360,7 @@ $.widget("ui.slider", $.ui.mouse, {
 			uiHash.value = this.values(index);
 			uiHash.values = this.values();
 		}
-		this._trigger("start", event, uiHash);
+		return this._trigger("start", event, uiHash);
 	},
 
 	_slide: function(event, index, newVal) {