From b6cda3dfeeca692fd14e5ecbde93286507e5c560 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Scott=20Gonz=C3=A1lez?= Date: Sun, 29 May 2011 19:41:57 -0400 Subject: [PATCH] Tooltip: Added some comments. --- ui/jquery.ui.tooltip.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) 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; } -- 2.39.5