From: Richard D. Worth <rdworth@gmail.com>
Date: Thu, 1 Apr 2010 19:08:20 +0000 (-0400)
Subject: Slider: added support for canceling start event. Fixes #5472 - slider: event start... 
X-Git-Tag: 1.8.1~5^2~43
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=354682e3ebba79e5b0ad014ea10c43f0a57a25e2;p=jquery-ui.git

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

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