From: Scott González <scott.gonzalez@gmail.com>
Date: Wed, 13 Jun 2012 09:18:48 +0000 (-0400)
Subject: Tooltip: When calling the content response method multiple times, only update the... 
X-Git-Tag: 1.9.0-beta.1~56
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=e8b623207abcae500ffb3860378543906cd4e2b2;p=jquery-ui.git

Tooltip: When calling the content response method multiple times, only update the content, don't do any other processing.
---

diff --git a/ui/jquery.ui.tooltip.js b/ui/jquery.ui.tooltip.js
index 1892d6555..752c95687 100644
--- a/ui/jquery.ui.tooltip.js
+++ b/ui/jquery.ui.tooltip.js
@@ -149,6 +149,14 @@ $.widget( "ui.tooltip", {
 			return;
 		}
 
+		// Content can be updated multiple times. If the tooltip already
+		// exists, then just update the content and bail.
+		var tooltip = this._find( target );
+		if ( tooltip.length ) {
+			tooltip.find( ".ui-tooltip-content" ).html( content );
+			return;
+		}
+
 		// if we have a title, clear it to prevent the native tooltip
 		// we have to check first to avoid defining a title if none exists
 		// (we don't want to cause an element to start matching [title])
@@ -164,15 +172,10 @@ $.widget( "ui.tooltip", {
 			}
 		}
 
-		// ajaxy tooltip can update an existing one
-		var tooltip = this._find( target );
-		if ( !tooltip.length ) {
-			tooltip = this._tooltip( target );
-			addDescribedBy( target, tooltip.attr( "id" ) );
-		}
+		tooltip = this._tooltip( target );
+		addDescribedBy( target, tooltip.attr( "id" ) );
 		tooltip.find( ".ui-tooltip-content" ).html( content );
 		tooltip
-			.stop( true )
 			.position( $.extend({
 				of: target
 			}, this.options.position ) )