Permalink
Browse files

Tooltip: Change the default items selector to exclude disabled elemen…

…ts. Fixes #8661 - Tooltip doesn't hide on disabled anchor element [IE only].
  • Loading branch information...
1 parent 94221c4 commit bd3a348776bf50da53d17f1574efd5538ce9de4f @scottgonzalez scottgonzalez committed Oct 13, 2012
Showing with 3 additions and 2 deletions.
  1. +1 −1 tests/unit/tooltip/tooltip_common.js
  2. +2 −1 ui/jquery.ui.tooltip.js
View
2 tests/unit/tooltip/tooltip_common.js
@@ -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",
View
3 ui/jquery.ui.tooltip.js
@@ -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",

0 comments on commit bd3a348

Please sign in to comment.