Skip to content

Commit daacbaf

Browse files
committed
Tooltip: Document why we don't use removeAttr
1 parent e2ca906 commit daacbaf

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

ui/jquery.ui.tooltip.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,10 @@ $.widget( "ui.tooltip", {
123123
// if we have a title, clear it to prevent the native tooltip
124124
// we have to check first to avoid defining a title if none exists
125125
// (we don't want to cause an element to start matching [title])
126-
// TODO: document why we don't use .removeAttr()
126+
127+
// We don't use removeAttr as that causes the native tooltip to show
128+
// up in IE (9 and below, didn't yet test 10). Happens only when removing
129+
// inside the mouseover handler.
127130
if ( target.is( "[title]" ) ) {
128131
target.attr( "title", "" );
129132
}

0 commit comments

Comments
 (0)