]> source.dussan.org Git - jquery-ui.git/commitdiff
Tooltip: Fix the accessible properties IE exposes
authorJörn Zaefferer <joern.zaefferer@gmail.com>
Wed, 9 May 2012 16:24:47 +0000 (18:24 +0200)
committerJörn Zaefferer <joern.zaefferer@gmail.com>
Wed, 9 May 2012 16:26:19 +0000 (18:26 +0200)
ui/jquery.ui.tooltip.js

index 8ddcbb7005aee601453307c3608777d58d7f886c..47a377bfd0a65e0dcb9e184fde1980cd46bb5257 100644 (file)
@@ -128,11 +128,15 @@ $.widget( "ui.tooltip", {
                // 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])
 
-               // We don't use removeAttr as that causes the native tooltip to show
-               // up in IE (9 and below, didn't yet test 10). Happens only when removing
-               // inside the mouseover handler.
+               // We use removeAttr only for key events, to allow IE to export the correct
+               // accessible attributes. For mouse events, set to empty string to avoid
+               // native tooltip showing up (happens only when removing inside mouseover).
                if ( target.is( "[title]" ) ) {
-                       target.attr( "title", "" );
+                       if ( event && event.type === "mouseover" ) {
+                               target.attr( "title", "" );
+                       } else {
+                               target.removeAttr( "title" );
+                       }
                }
 
                // ajaxy tooltip can update an existing one