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/29 19:11:22 UTC

svn commit: r480684 - in /incubator/xap/trunk/src/xap: bridges/xap/ListBoxBridge.js widgets/ListBox.js

Author: mturyn
Date: Wed Nov 29 11:11:16 2006
New Revision: 480684

URL: http://svn.apache.org/viewvc?view=rev&rev=480684
Log:
Changed value is now properly written back to the element.

Modified:
    incubator/xap/trunk/src/xap/bridges/xap/ListBoxBridge.js
    incubator/xap/trunk/src/xap/widgets/ListBox.js

Modified: incubator/xap/trunk/src/xap/bridges/xap/ListBoxBridge.js
URL: http://svn.apache.org/viewvc/incubator/xap/trunk/src/xap/bridges/xap/ListBoxBridge.js?view=diff&rev=480684&r1=480683&r2=480684
==============================================================================
--- incubator/xap/trunk/src/xap/bridges/xap/ListBoxBridge.js (original)
+++ incubator/xap/trunk/src/xap/bridges/xap/ListBoxBridge.js Wed Nov 29 11:11:16 2006
@@ -40,6 +40,18 @@
 }
 
 
+xap.bridges.xap.ListBoxBridge.prototype.init = function() {
+	xap.bridges.dojo.DojoWidgetBridge.prototype.init.call(this);
+	//add listener last so we don't fire for the initial set
+	dojo.event.connect(this.getPeer(), "onchange",this,"onChange");			
+}
+
+xap.bridges.xap.ListBoxBridge.prototype.onChange = function(event){
+	//xap.util.Utils.interrogate(event.originalTarget) ;
+	this.writeBackAttribute("value",event.originalTarget.value);	
+}
+
+
 xap.bridges.xap.ListBoxBridge.prototype.addChild = function(childHandler) {
 	
 	var elementListItem = childHandler.getElement();

Modified: incubator/xap/trunk/src/xap/widgets/ListBox.js
URL: http://svn.apache.org/viewvc/incubator/xap/trunk/src/xap/widgets/ListBox.js?view=diff&rev=480684&r1=480683&r2=480684
==============================================================================
--- incubator/xap/trunk/src/xap/widgets/ListBox.js (original)
+++ incubator/xap/trunk/src/xap/widgets/ListBox.js Wed Nov 29 11:11:16 2006
@@ -17,7 +17,7 @@
 dojo.lang.extend(xap.widgets.ListBox, {
 // If size==1, get a combo box, which we don't want, so
 // start with size 2, increment if necessary as we add children:
-		templateString: '<select dojoAttachPoint="select" size="2"></select>',
+		templateString: '<select dojoAttachPoint="select" dojoAttachEvent="onkeyup;onchange;onmouseout" size="2"></select>',
 		templateCssPath: null,
 		widgetType: "ListBox",
 		isContainer: true,