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 jm...@apache.org on 2007/09/12 23:54:20 UTC
svn commit: r575131 -
/incubator/xap/trunk/codebase/src/xap/widgets/ListBox.js
Author: jmargaris
Date: Wed Sep 12 16:54:19 2007
New Revision: 575131
URL: http://svn.apache.org/viewvc?rev=575131&view=rev
Log:
getting rid of hardcoded margin/padding on listbox,
fixing size issue
Modified:
incubator/xap/trunk/codebase/src/xap/widgets/ListBox.js
Modified: incubator/xap/trunk/codebase/src/xap/widgets/ListBox.js
URL: http://svn.apache.org/viewvc/incubator/xap/trunk/codebase/src/xap/widgets/ListBox.js?rev=575131&r1=575130&r2=575131&view=diff
==============================================================================
--- incubator/xap/trunk/codebase/src/xap/widgets/ListBox.js (original)
+++ incubator/xap/trunk/codebase/src/xap/widgets/ListBox.js Wed Sep 12 16:54:19 2007
@@ -44,7 +44,6 @@
templateString: '<select dojoAttachPoint="select" '
+ 'dojoAttachEvent="onchange;onkeydown;onmouseup"'
- + 'style="margin:0px;padding:0px"'
+ 'size="2"></select>',
templateCssPath: null,
widgetType: "ListBox",
@@ -81,7 +80,7 @@
}
else{
- this.select.size = this.select.options.length;
+ this.select.size=this.select.options.length<2?2:this.select.options.length;
}
},