added CSS formatting for sizes, so we don't get decimals for pixel sizes

svn changeset:5536/svn branch:trunk

diff --git a/src/com/itmill/toolkit/ui/AbstractComponent.java b/src/com/itmill/toolkit/ui/AbstractComponent.java
index b4a33468b3239cfa488c3fb304ef2a4774e5b6ea..a80d79cb56e05e239879ee373d98e1e0c65bdd59 100644 (file)
--- a/src/com/itmill/toolkit/ui/AbstractComponent.java
+++ b/src/com/itmill/toolkit/ui/AbstractComponent.java
@@ -604,12 +604,10 @@ public abstract class AbstractComponent implements Component, MethodEventSource
             if (isVisible()) {
 
                 if (getHeight() >= 0) {
-                    target.addAttribute("height", "" + getHeight()
-                            + UNIT_SYMBOLS[getHeightUnits()]);
+                    target.addAttribute("height", "" + getCSSHeight());
                 }
                 if (getWidth() >= 0) {
-                    target.addAttribute("width", "" + getWidth()
-                            + UNIT_SYMBOLS[getWidthUnits()]);
+                    target.addAttribute("width", "" + getCSSWidth());
                 }
 
                 if (styles != null && styles.size() > 0) {
@@ -655,6 +653,32 @@ public abstract class AbstractComponent implements Component, MethodEventSource
         repaintRequestListenersNotified = false;
     }
 
+    /**
+     * Build CSS compatible string representation of height.
+     * 
+     * @return CSS height
+     */
+    private String getCSSHeight() {
+        if (getHeightUnits() == UNITS_PIXELS) {
+            return ((int) getHeight()) + UNIT_SYMBOLS[getHeightUnits()];
+        } else {
+            return getHeight() + UNIT_SYMBOLS[getHeightUnits()];
+        }
+    }
+
+    /**
+     * Build CSS compatible string representation of width.
+     * 
+     * @return CSS width
+     */
+    private String getCSSWidth() {
+        if (getWidthUnits() == UNITS_PIXELS) {
+            return ((int) getWidth()) + UNIT_SYMBOLS[getWidthUnits()];
+        } else {
+            return getWidth() + UNIT_SYMBOLS[getWidthUnits()];
+        }
+    }
+
     /**
      * Paints any needed component-specific things to the given UIDL stream. The
      * more general {@link #paint(PaintTarget)} method handles all general