Changeset 5077

Timestamp:

07/10/08 12:20:23 (6 months ago)

Author:

matti.tahvonen@…

Message:

partial fix for #1902. FF seems to bug some pixels still (too narrow).

Location:

trunk/src/com/itmill/toolkit

Files:

• terminal/gwt/client/ui/ITextField.java (modified) (4 diffs)
• tests/tickets/Ticket1902.java (modified) (3 diffs)

trunk/src/com/itmill/toolkit/terminal/gwt/client/ui/ITextField.java

@@ -13 +13 @@
 import com.google.gwt.user.client.ui.Widget;
 import com.itmill.toolkit.terminal.gwt.client.ApplicationConnection;
+import com.itmill.toolkit.terminal.gwt.client.ContainerResizedListener;
 import com.itmill.toolkit.terminal.gwt.client.Paintable;
 import com.itmill.toolkit.terminal.gwt.client.Tooltip;
 import com.itmill.toolkit.terminal.gwt.client.UIDL;
+import com.itmill.toolkit.terminal.gwt.client.Util;
 
 /**
@@ -24 +26 @@
  */
 public class ITextField extends TextBoxBase implements Paintable, Field,
-        ChangeListener, FocusListener {
+        ChangeListener, FocusListener, ContainerResizedListener {
 
     /**
@@ -40 +42 @@
 
     private boolean immediate = false;
+    private float proportionalHeight = -1;
+    private float proportionalWidth = -1;
+    private int extraHorizontalPixels = -1;
+    private int extraVerticalPixels = -1;
 
     public ITextField() {
@@ -119 +125 @@
     }-*/;
 
+    public void setHeight(String height) {
+        if (height != null && height.indexOf("%") > 0) {
+            // special handling for proportional height
+            proportionalHeight = Float.parseFloat(height.substring(0, height
+                    .indexOf("%"))) / 100;
+            iLayout();
+        } else {
+            super.setHeight(height);
+            proportionalHeight = -1;
+        }
+    }
+
+    public void setWidth(String width) {
+        if (width != null && width.indexOf("%") > 0) {
+            // special handling for proportional w
+            proportionalWidth = Float.parseFloat(width.substring(0, width
+                    .indexOf("%"))) / 100;
+            iLayout();
+        } else {
+            super.setHeight(width);
+            proportionalWidth = -1;
+        }
+    }
+
+    public void iLayout() {
+        if (proportionalWidth >= 0) {
+            int availPixels = (int) (DOM.getElementPropertyInt(DOM
+                    .getParent(getElement()), "clientWidth") * proportionalWidth);
+            availPixels -= getExtraHorizontalPixels();
+            super.setWidth(availPixels + "px");
+        }
+        if (proportionalHeight >= 0) {
+            int availPixels = (int) (DOM.getElementPropertyInt(DOM
+                    .getParent(getElement()), "clientHeight") * proportionalHeight);
+            availPixels -= getExtraVerticalPixels();
+            super.setHeight(availPixels + "px");
+        }
+    }
+
+    /**
+     * @return space used by components paddings and borders
+     */
+    private int getExtraHorizontalPixels() {
+        if (extraHorizontalPixels < 0) {
+            detectExtraSizes();
+        }
+        return extraHorizontalPixels;
+    }
+
+    /**
+     * @return space used by components paddings and borders
+     */
+    private int getExtraVerticalPixels() {
+        if (extraVerticalPixels < 0) {
+            detectExtraSizes();
+        }
+        return extraVerticalPixels;
+    }
+
+    /**
+     * Detects space used by components paddings and borders. Used when
+     * relational size are used.
+     */
+    private void detectExtraSizes() {
+        Element clone = Util.cloneNode(getElement(), false);
+        DOM.setElementAttribute(clone, "id", "");
+        DOM.setStyleAttribute(clone, "visibility", "hidden");
+        DOM.setStyleAttribute(clone, "position", "absolute");
+        DOM.setStyleAttribute(clone, "width", "0");
+        DOM.setStyleAttribute(clone, "height", "0");
+        DOM.appendChild(DOM.getParent(getElement()), clone);
+        extraHorizontalPixels = DOM.getElementPropertyInt(clone, "offsetWidth");
+        extraVerticalPixels = DOM.getElementPropertyInt(clone, "offsetHeight");
+        DOM.removeChild(DOM.getParent(getElement()), clone);
+    }
+
 }

trunk/src/com/itmill/toolkit/tests/tickets/Ticket1902.java

@@ -36 +36 @@
 
         // 100% wide component
-        Button b2 = new Button("100% wide button");
+        TextField b2 = new TextField("100% wide field");
         mainLayout.addComponent(b2);
         b2.setWidth("100%");
@@ -58 +58 @@
         lo2.setStyleName("red-background");
         mainLayout.addComponent(lo2);
-        lo2.setWidth(400);
+        lo2.setWidth("50%");
         lo2.setHeight(200);
 
@@ -68 +68 @@
         lo2.addComponent(tf2);
         tf2.setWidth("100%");
+        // tf2 = new TextField("50% wide, 100% height textfield"); // does not
+        // work with caption (10.7.2008 mac hosted mode) due layouts are broken
+        // in trunk
+        tf2 = new TextField();
+        tf2.setRows(2); // trigger textArea impl.
+        tf2.setHeight("100%");
+        tf2.setWidth("50%");
+        lo2.addComponent(tf2);
     }
 }