You are viewing a plain text version of this content. The canonical link for it is here.
Posted to xap-commits@incubator.apache.org by mt...@apache.org on 2006/11/13 19:39:01 UTC

svn commit: r474460 - /incubator/xap/trunk/src/xap/bridges/xap/ButtonBridge.js

Author: mturyn
Date: Mon Nov 13 11:39:00 2006
New Revision: 474460

URL: http://svn.apache.org/viewvc?view=rev&rev=474460
Log:
Now handles mouse out in mid-click (mouse over, mouse down, mouse out, mouse up) correctly.

Modified:
    incubator/xap/trunk/src/xap/bridges/xap/ButtonBridge.js

Modified: incubator/xap/trunk/src/xap/bridges/xap/ButtonBridge.js
URL: http://svn.apache.org/viewvc/incubator/xap/trunk/src/xap/bridges/xap/ButtonBridge.js?view=diff&rev=474460&r1=474459&r2=474460
==============================================================================
--- incubator/xap/trunk/src/xap/bridges/xap/ButtonBridge.js (original)
+++ incubator/xap/trunk/src/xap/bridges/xap/ButtonBridge.js Mon Nov 13 11:39:00 2006
@@ -30,6 +30,11 @@
 	this.setPeer(new xap.widgets.Button());
 	this.getPeer().init();
 	dojo.event.connect(this.getRootDomNode(), "onclick", this, "onclick");
+	// Need to be able to handle
+	// 1.) mousedown on button  2.) mouseout  3.) mouseup
+	// properly---capture for these events is already set
+	// in aButton.onMouseDown, unset in aButton.onMouseUp
+	dojo.event.connectBefore(this.getPeer(), "_onmouseup", this, "onMouseUp");			
 }
 
 xap.bridges.xap.ButtonBridge.prototype.onclick = function() {