From: Richard Worth <rdworth@gmail.com>
Date: Fri, 16 Jan 2009 04:06:51 +0000 (+0000)
Subject: Fixed #3077 (Selectable cannot be selected by click)
X-Git-Tag: 1.6rc5~8
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=2809d50800dc0da1d43766f942e2fb9330d1e4c6;p=jquery-ui.git

Fixed #3077 (Selectable cannot be selected by click)
---

diff --git a/ui/ui.selectable.js b/ui/ui.selectable.js
index d8f86a406..8228c95ce 100644
--- a/ui/ui.selectable.js
+++ b/ui/ui.selectable.js
@@ -33,8 +33,8 @@ $.widget("ui.selectable", $.extend({}, $.ui.mouse, {
 					$element: $this,
 					left: pos.left,
 					top: pos.top,
-					right: pos.left + $this.width(),
-					bottom: pos.top + $this.height(),
+					right: pos.left + $this.outerWidth(),
+					bottom: pos.top + $this.outerHeight(),
 					startselected: false,
 					selected: $this.hasClass('ui-selected'),
 					selecting: $this.hasClass('ui-selecting'),
@@ -105,11 +105,21 @@ $.widget("ui.selectable", $.extend({}, $.ui.mouse, {
 			}
 		});
 
-		var isSelectee = false;
 		$(event.target).parents().andSelf().each(function() {
-			if($.data(this, "selectable-item")) isSelectee = true;
+			var selectee = $.data(this, "selectable-item");
+			if (selectee) {
+				selectee.$element.removeClass("ui-unselecting").addClass('ui-selecting');
+				selectee.unselecting = false;
+				selectee.selecting = true;
+				selectee.selected = true;
+				// selectable SELECTING callback
+				self._trigger("selecting", event, {
+					selecting: selectee.element
+				});
+				return false;
+			}
 		});
-		return this.options.keyboard ? !isSelectee : true;
+
 	},
 
 	_mouseDrag: function(event) {
@@ -238,7 +248,7 @@ $.extend($.ui.selectable, {
 		autoRefresh: true,
 		cancel: ":input,option",
 		delay: 0,
-		distance: 1,
+		distance: 0,
 		filter: '*',
 		tolerance: 'touch'
 	}