svn commit: r494618 - /incubator/xap/trunk/src/xap/bridges/basic/AbstractWidgetBridge.js

[jm...@apache.org]

Author: jmargaris
Date: Tue Jan  9 14:13:25 2007
New Revision: 494618

URL: http://svn.apache.org/viewvc?view=rev&rev=494618
Log:
Fix for horrible tooltip performance, setup was leaving a dangling
listener for mouseOver on the entire window

Modified:
    incubator/xap/trunk/src/xap/bridges/basic/AbstractWidgetBridge.js

Modified: incubator/xap/trunk/src/xap/bridges/basic/AbstractWidgetBridge.js
URL: http://svn.apache.org/viewvc/incubator/xap/trunk/src/xap/bridges/basic/AbstractWidgetBridge.js?view=diff&rev=494618&r1=494617&r2=494618
==============================================================================
--- incubator/xap/trunk/src/xap/bridges/basic/AbstractWidgetBridge.js (original)
+++ incubator/xap/trunk/src/xap/bridges/basic/AbstractWidgetBridge.js Tue Jan  9 14:13:25 2007
@@ -712,13 +712,14 @@
 	var tooltip = dojo.widget.createWidget('richtexttooltip',{caption:text},tmp);
 	tooltip.domNode.enabled=false;	
 	
+	//in postCreate or fillInTemplate dojo tried to connect to "connectNode"
+	//but it was udefined and ended up connecting to window, so disconnect here
+	dojo.event.disconnect(tooltip.connectNode, "onmouseover", tooltip, "onMouseOver");
+	
 	//we have to set this connection up ourselves because it was not available
 	//in postCreate()
 	tooltip.connectNode = this.getRootDomNode();
-	dojo.event.connect(tooltip.connectNode, "onmouseover", tooltip, "onMouseOver");
-	// Remove tooltip as soon as mouse is out:
-	dojo.event.connect(tooltip.connectNode, "onmouseout", tooltip, "hide");	
-	
+	dojo.event.connect(tooltip.connectNode, "onmouseover", tooltip, "onMouseOver"); 
 }
 
 xap.bridges.basic.AbstractWidgetBridge.prototype.setFocusedAttribute = function( text ){