]> source.dussan.org Git - jquery-ui.git/commitdiff
Tooltip: Follow the standard appendTo logic
authorScott González <scott.gonzalez@gmail.com>
Mon, 23 Mar 2015 23:09:50 +0000 (19:09 -0400)
committerScott González <scott.gonzalez@gmail.com>
Thu, 26 Mar 2015 11:35:20 +0000 (07:35 -0400)
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

ui/tooltip.js

index ae10177223b24c76fceea923fba1c4f4a818174d..841e877cc2bee7a1b3ac1bef55461338d5a03aca 100644 (file)
@@ -428,7 +428,7 @@ $.widget( "ui.tooltip", {
                this._addClass( content, "ui-tooltip-content" );
                this._addClass( tooltip, "ui-tooltip", "ui-widget ui-widget-content" );
 
-               tooltip.appendTo( this.document[ 0 ].body );
+               tooltip.appendTo( this._appendTo( element ) );
 
                return this.tooltips[ id ] = {
                        element: element,
@@ -446,6 +446,16 @@ $.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;