Skip to content

Commit

Permalink
Tooltip: Change the default items selector to exclude disabled elemen…
Browse files Browse the repository at this point in the history
…ts. Fixes #8661 - Tooltip doesn't hide on disabled anchor element [IE only].
  • Loading branch information
scottgonzalez committed Oct 13, 2012
1 parent 94221c4 commit bd3a348
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion tests/unit/tooltip/tooltip_common.js
Expand Up @@ -3,7 +3,7 @@ TestHelpers.commonWidgetTests( "tooltip", {
content: function() {},
disabled: false,
hide: true,
items: "[title]",
items: "[title]:not([disabled])",
position: {
my: "left+15 center",
at: "right center",
Expand Down
3 changes: 2 additions & 1 deletion ui/jquery.ui.tooltip.js
Expand Up @@ -49,7 +49,8 @@ $.widget( "ui.tooltip", {
return $( this ).attr( "title" );
},
hide: true,
items: "[title]",
// Disabled elements have inconsistent behavior across browsers (#8661)
items: "[title]:not([disabled])",
position: {
my: "left+15 center",
at: "right center",
Expand Down

0 comments on commit bd3a348

Please sign in to comment.