From: Scott González <scott.gonzalez@gmail.com>
Date: Sun, 29 May 2011 23:41:57 +0000 (-0400)
Subject: Tooltip: Added some comments.
X-Git-Tag: 1.9m6~111^2~3
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=b6cda3dfeeca692fd14e5ecbde93286507e5c560;p=jquery-ui.git

Tooltip: Added some comments.
---

diff --git a/ui/jquery.ui.tooltip.js b/ui/jquery.ui.tooltip.js
index 7722e6472..8aba5974a 100644
--- a/ui/jquery.ui.tooltip.js
+++ b/ui/jquery.ui.tooltip.js
@@ -83,6 +83,9 @@ $.widget( "ui.tooltip", {
 		}
 
 		// if we have a title, clear it to prevent the native tooltip
+		// we do this before the disabled check to prevent native tooltips
+		// even when disabled
+		// TODO: the above doesn't work since ._bind() does a disabled check
 		// we have to check first to avoid defining a title if none exists
 		// (we don't want to cause an element to start matching [title])
 		// TODO: document why we don't use .removeAttr()
@@ -90,7 +93,6 @@ $.widget( "ui.tooltip", {
 			target.attr( "title", "" );
 		}
 
-		// TODO: why is this check after we clear the title?
 		if ( this.options.disabled ) {
 			return;
 		}