From 977b33625be3dd0959bf5b1272f2f192471794cf Mon Sep 17 00:00:00 2001 From: =?utf8?q?Scott=20Gonz=C3=A1lez?= Date: Mon, 23 Mar 2015 19:09:50 -0400 Subject: [PATCH] Tooltip: Follow the standard appendTo logic Even though there's no reason to ever configure the parent element via an `appendTo` option, following the standard logic is useful for scrollable elements and native dialogs. Fixes #10739 Closes gh-1517 (cherry picked from commit 8cf98798a58afb7197bce0d6ca1c5b658ae04011) --- ui/tooltip.js | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/ui/tooltip.js b/ui/tooltip.js index 8408f6781..23446ae65 100644 --- a/ui/tooltip.js +++ b/ui/tooltip.js @@ -424,7 +424,7 @@ return $.widget( "ui.tooltip", { .addClass( "ui-tooltip-content" ) .appendTo( tooltip ); - tooltip.appendTo( this.document[0].body ); + tooltip.appendTo( this._appendTo( element ) ); return this.tooltips[ id ] = { element: element, @@ -442,6 +442,16 @@ return $.widget( "ui.tooltip", { delete this.tooltips[ tooltip.attr( "id" ) ]; }, + _appendTo: function( target ) { + var element = target.closest( ".ui-front, dialog" ); + + if ( !element.length ) { + element = this.document[ 0 ].body; + } + + return element; + }, + _destroy: function() { var that = this; -- 2.39.5