Use .has() instead of .closest() to fix 'unfocus' event in jQuery 1.4…

….4 (Thanks m4olivei)
qTip2 · Feb 27, 2012 · 384c70a · 384c70a
1 parent dda65be
commit 384c70a
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 17 deletions.
diff --git a/dist/jquery.qtip.basic.js b/dist/jquery.qtip.basic.js
@@ -9,7 +9,7 @@
 *   http://en.wikipedia.org/wiki/MIT_License
 *   http://en.wikipedia.org/wiki/GNU_General_Public_License
 *
-* Date: Mon Feb 27 01:41:07 2012 +0000
+* Date: Mon Feb 27 02:03:04 2012 +0000
 */
 
 /*jslint browser: true, onevar: true, undef: true, nomen: true, bitwise: true, regexp: true, newcap: true, immed: true, strict: true */
@@ -557,12 +557,12 @@ function QTip(target, options, id, attr)
 		// Hide tooltip on document mousedown if unfocus events are enabled
 		if(('' + options.hide.event).indexOf('unfocus') > -1) {
 			posOptions.container.closest('html').bind('mousedown'+namespace, function(event) {
-				var $target = $(event.target),
+				var elem = $(event.target),
 					enabled = !tooltip.hasClass(disabled) && tooltip.is(':visible'),
-					isAncestor = $target.parents(selector).filter(tooltip[0]).length > 0;
+					isAncestor = elem.parents(selector).filter(tooltip[0]).length > 0;
 
-				if($target[0] !== tooltip[0] && !isAncestor &&
-					!$target.closest(target).length && !$target.attr('disabled')
+				if(elem[0] !== target[0] && elem[0] !== tooltip[0] && !isAncestor &&
+					!target.has(elem[0]).length && !elem.attr('disabled')
 				) {
 					self.hide(event);
 				}

diff --git a/dist/jquery.qtip.css b/dist/jquery.qtip.css
@@ -9,7 +9,7 @@
 *   http://en.wikipedia.org/wiki/MIT_License
 *   http://en.wikipedia.org/wiki/GNU_General_Public_License
 *
-* Date: Mon Feb 27 01:41:07 2012 +0000
+* Date: Mon Feb 27 02:03:04 2012 +0000
 */
 
 /* Core qTip styles */

diff --git a/dist/jquery.qtip.js b/dist/jquery.qtip.js
@@ -9,7 +9,7 @@
 *   http://en.wikipedia.org/wiki/MIT_License
 *   http://en.wikipedia.org/wiki/GNU_General_Public_License
 *
-* Date: Mon Feb 27 01:41:07 2012 +0000
+* Date: Mon Feb 27 02:03:04 2012 +0000
 */
 
 /*jslint browser: true, onevar: true, undef: true, nomen: true, bitwise: true, regexp: true, newcap: true, immed: true, strict: true */
@@ -557,12 +557,12 @@ function QTip(target, options, id, attr)
 		// Hide tooltip on document mousedown if unfocus events are enabled
 		if(('' + options.hide.event).indexOf('unfocus') > -1) {
 			posOptions.container.closest('html').bind('mousedown'+namespace, function(event) {
-				var $target = $(event.target),
+				var elem = $(event.target),
 					enabled = !tooltip.hasClass(disabled) && tooltip.is(':visible'),
-					isAncestor = $target.parents(selector).filter(tooltip[0]).length > 0;
+					isAncestor = elem.parents(selector).filter(tooltip[0]).length > 0;
 
-				if($target[0] !== tooltip[0] && !isAncestor &&
-					!$target.closest(target).length && !$target.attr('disabled')
+				if(elem[0] !== target[0] && elem[0] !== tooltip[0] && !isAncestor &&
+					!target.has(elem[0]).length && !elem.attr('disabled')
 				) {
 					self.hide(event);
 				}

diff --git a/dist/jquery.qtip.min.js b/dist/jquery.qtip.min.js
diff --git a/src/core.js b/src/core.js
@@ -485,12 +485,12 @@ function QTip(target, options, id, attr)
 		// Hide tooltip on document mousedown if unfocus events are enabled
 		if(('' + options.hide.event).indexOf('unfocus') > -1) {
 			posOptions.container.closest('html').bind('mousedown'+namespace, function(event) {
-				var $target = $(event.target),
+				var elem = $(event.target),
 					enabled = !tooltip.hasClass(disabled) && tooltip.is(':visible'),
-					isAncestor = $target.parents(selector).filter(tooltip[0]).length > 0;
+					isAncestor = elem.parents(selector).filter(tooltip[0]).length > 0;
 
-				if($target[0] !== tooltip[0] && !isAncestor &&
-					!$target.closest(target).length && !$target.attr('disabled')
+				if(elem[0] !== target[0] && elem[0] !== tooltip[0] && !isAncestor &&
+					!target.has(elem[0]).length && !elem.attr('disabled')
 				) {
 					self.hide(event);
 				}