Skip to content
Permalink
Browse files

Tooltip: Document why we don't use removeAttr

  • Loading branch information
jzaefferer committed Feb 19, 2012
1 parent e2ca906 commit daacbaf22180cc269fb730d8652d5ee72389a6d0
Showing with 4 additions and 1 deletion.
  1. +4 −1 ui/jquery.ui.tooltip.js
@@ -123,7 +123,10 @@ $.widget( "ui.tooltip", {
// if we have a title, clear it to prevent the native 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])
// TODO: document why we don't use .removeAttr()

// 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.
if ( target.is( "[title]" ) ) {
target.attr( "title", "" );
}

0 comments on commit daacbaf

Please sign in to comment.
You can’t perform that action at this time.