From: Jack Hsu <jack.hsu@gmail.com>
Date: Wed, 12 May 2010 14:51:13 +0000 (+0800)
Subject: Selectable: modified _mouseStart function to unselect event.target if the Ctrl/Cmd... 
X-Git-Tag: 1.8.2~9
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=5b4c04acb5cff032afae437c343239ee9bf4e04f;p=jquery-ui.git

Selectable: modified _mouseStart function to unselect event.target if the Ctrl/Cmd key is pressed, and target is selected. Fixed #4293 - Ctrl + Click on selected list item does not deselect
---

diff --git a/ui/jquery.ui.selectable.js b/ui/jquery.ui.selectable.js
index a14f720b7..d679d1018 100644
--- a/ui/jquery.ui.selectable.js
+++ b/ui/jquery.ui.selectable.js
@@ -121,12 +121,15 @@ $.widget("ui.selectable", $.ui.mouse, {
 		$(event.target).parents().andSelf().each(function() {
 			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, {
+				var doSelect = !event.metaKey || !selectee.$element.hasClass('ui-selected');
+				selectee.$element
+					.removeClass(doSelect ? "ui-unselecting" : "ui-selected")
+					.addClass(doSelect ? "ui-selecting" : "ui-unselecting");
+				selectee.unselecting = !doSelect;
+				selectee.selecting = doSelect;
+				selectee.selected = doSelect;
+				// selectable UNSELECTING callback
+				self._trigger(doSelect ? "selecting" : "unselecting", event, {
 					selecting: selectee.element
 				});
 				return false;