fixed width with combobox, deprecated setColumns

svn changeset:4047/svn branch:trunk

1 files changed, 10 insertions, 5 deletions

diff --git a/src/com/itmill/toolkit/ui/Select.java b/src/com/itmill/toolkit/ui/Select.java
index 791b2d463c..41b203f4b5 100644
--- a/src/com/itmill/toolkit/ui/Select.java
+++ b/src/com/itmill/toolkit/ui/Select.java
@@ -126,11 +126,6 @@ public class Select extends AbstractSelect implements AbstractSelect.Filtering {
             }
         }
 
-        // Adds the number of columns
-        if (columns != 0) {
-            target.addAttribute("cols", columns);
-        }
-
         // Constructs selected keys array
         String[] selectedKeys;
         if (isMultiSelect()) {
@@ -425,10 +420,15 @@ public class Select extends AbstractSelect implements AbstractSelect.Filtering {
     }
 
     /**
+     * Note, one should use more generic setWidth(String) method instead of
+     * this. This now days actually converts columns to width with em css unit.
+     * 
      * Sets the number of columns in the editor. If the number of columns is set
      * 0, the actual number of displayed columns is determined implicitly by the
      * adapter.
      * 
+     * @deprecated
+     * 
      * @param columns
      *                the number of columns to set.
      */
@@ -438,10 +438,15 @@ public class Select extends AbstractSelect implements AbstractSelect.Filtering {
         }
         if (this.columns != columns) {
             this.columns = columns;
+            setWidth(columns, Select.UNITS_EM);
             requestRepaint();
         }
     }
 
+    /**
+     * @deprecated see setter function
+     * @return
+     */
     public int getColumns() {
         return columns;
     }