Changeset 10534

Timestamp:

09/17/07 07:05:44 (14 months ago)

Author:

bill

Message:

Remove some unnecessary code that messing up diji.place()'s detection of
the size of the dropdown (messing up Combobox drop down sizing). Refs #3058.

Location:

dijit/trunk/form

Files:

• ComboBox.js (modified) (1 diff)
• _DropDownTextBox.js (modified) (1 diff)

dijit/trunk/form/ComboBox.js

@@ -470 +470 @@
                 //      Focus-less div based menu for internal use in ComboBox
 
-                templateString:"<div class='dijitMenu' dojoAttachEvent='onclick,onmouseover,onmouseout' tabIndex='-1' style='display:none; position:absolute; overflow:\"auto\";'>"
+                templateString:"<div class='dijitMenu' dojoAttachEvent='onclick,onmouseover,onmouseout' tabIndex='-1' style='overflow:\"auto\";'>"
                                 +"<div class='dijitMenuItem' dojoAttachPoint='previousButton'></div>"
                                 +"<div class='dijitMenuItem' dojoAttachPoint='nextButton'></div>"

dijit/trunk/form/_DropDownTextBox.js

@@ -138 +138 @@
                         if(childs.length){
                                 var visibleCount = Math.min(childs.length,this.maxListLength);
-                                with(this._popupWidget.domNode.style){
-                                        // trick to get the dimensions of the popup
-                                        // TODO: doesn't dijit.popup.open() do this automatically?
-                                        display="";
-                                        width="";
-                                        height="";
-                                }
                                 this._arrowPressed();
                                 // hide the tooltip