library-mirrors/jquery-ui
1
0
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2024-11-21 11:04:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Tooltip: handle removal of elements with delegated tooltips. Fixed #8646 - Delegated tooltips don't close when the tooltipped element is removed

Browse Source
This commit is contained in:
Andrew Couch 2012-10-15 16:15:36 -04:00 committed by Jörn Zaefferer
parent 6b48ef5eca
commit 3b2d1e7736
3 changed files with 28 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
tests/unit/tooltip/tooltip.html
View File

@ -43,6 +43,7 @@
<input title="inputtitle">
<span id="multiple-describedby" aria-describedby="fixture-span" title="...">aria-describedby</span>
<span id="fixture-span" title="title-text">span</span>
<span id="contains-tooltipped"><span id="contained-tooltipped" title="foobar">baz</span></span>
</div>
</div>

13
tests/unit/tooltip/tooltip_core.js
View File

@ -44,4 +44,17 @@ test( "accessibility", function() {
equal( element.attr( "title" ), "...", "title restored when closed" );
});
test( "delegated removal", function() {
expect( 2 );
var container = $( "#contains-tooltipped" ).tooltip(),
element = $( "#contained-tooltipped" );
element.trigger( "mouseover" );
equal( $( ".ui-tooltip" ).length, 1 );
container.empty();
equal( $( ".ui-tooltip" ).length, 0 );
});
}( jQuery ) );

16
ui/jquery.ui.tooltip.js vendored
View File

@ -207,6 +207,7 @@ $.widget( "ui.tooltip", {
_open: function( event, target, content ) {
var tooltip, positionOption, events;
if ( !content ) {
return;
}
@ -268,6 +269,9 @@ $.widget( "ui.tooltip", {
fakeEvent.currentTarget = target[0];
this.close( fakeEvent, true );
}
},
remove: function( event ) {
this._removeTooltip( tooltip );
}
};
if ( !event || event.type === "mouseover" ) {
@ -299,12 +303,15 @@ $.widget( "ui.tooltip", {
tooltip.stop( true );
this._hide( tooltip, this.options.hide, function() {
$( this ).remove();
delete that.tooltips[ this.id ];
that._removeTooltip( $( this ) );
});
target.removeData( "tooltip-open" );
this._off( target, "mouseleave focusout keyup" );
// Remove 'remove' binding only on delegated targets
if ( target[0] !== this.element[0] ) {
this._off( target, "remove" );
}
this._off( this.document, "mousemove" );
if ( event && event.type === "mouseleave" ) {
@ -344,6 +351,11 @@ $.widget( "ui.tooltip", {
return id ? $( "#" + id ) : $();
},
_removeTooltip: function( tooltip ) {
tooltip.remove();
delete this.tooltips[ tooltip.attr( "id" ) ];
},
_destroy: function() {
var that = this;