From: Scott González Date: Thu, 14 Jun 2012 14:06:21 +0000 (-0400) Subject: Tooltip: Reposition tracking tooltips when they gain focus. X-Git-Tag: 1.9.0-beta.1~42 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=d740e450aa7bd8a4f254f0268d74636d5920c60a;p=jquery-ui.git Tooltip: Reposition tracking tooltips when they gain focus. --- diff --git a/ui/jquery.ui.tooltip.js b/ui/jquery.ui.tooltip.js index 2c80f328c..5ef5051af 100644 --- a/ui/jquery.ui.tooltip.js +++ b/ui/jquery.ui.tooltip.js @@ -118,8 +118,18 @@ $.widget( "ui.tooltip", { target = $( event ? event.target : this.element ) .closest( this.options.items ); - // if ui-tooltip-id exists, then the tooltip is already open - if ( !target.length || target.data( "ui-tooltip-id" ) ) { + // No element to show a tooltip for + if ( !target.length ) { + return; + } + + // If the tooltip is open and we're tracking then reposition the tooltip. + // This makes sure that a tracking tooltip doesn't obscure a focused element + // if the user was hovering when the element gained focused. + if ( this.options.track && target.data( "ui-tooltip-id" ) ) { + this._find( target ).position( $.extend({ + of: target + }, this.options.position ) ); return; } @@ -250,9 +260,7 @@ $.widget( "ui.tooltip", { target.removeData( "tooltip-open" ); this._off( target, "mouseleave focusout keyup" ); - - // TODO use _off - this.document.unbind( "mousemove.tooltip" ); + this._off( this.document, "mousemove" ); this.closing = true; this._trigger( "close", event, { tooltip: tooltip } );