From: Artur Signell <artur.signell@itmill.com>
Date: Fri, 19 Sep 2008 08:51:03 +0000 (+0000)
Subject: Fix for #1966 - OrderedLayout vertical alignment
X-Git-Tag: 6.7.0.beta1~4131
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=b88fc7851e20a5ad6a81d88304051d62c31a1eca;p=vaadin-framework.git

Fix for #1966 - OrderedLayout vertical alignment

svn changeset:5459/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ui/IOrderedLayout.java b/src/com/itmill/toolkit/terminal/gwt/client/ui/IOrderedLayout.java
index f28218a84a..7dea7ec1da 100644
--- a/src/com/itmill/toolkit/terminal/gwt/client/ui/IOrderedLayout.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ui/IOrderedLayout.java
@@ -381,9 +381,16 @@ public class IOrderedLayout extends Panel implements Container,
         // Constuct base DOM-structure and clean any already attached
         // widgetwrappers from DOM.
         if (tableMode) {
-            final String structure = "<table cellspacing=\"0\" cellpadding=\"0\"><tbody>"
+            String structure = "<table cellspacing=\"0\" cellpadding=\"0\"";
+
+            if (orientationMode == ORIENTATION_HORIZONTAL) {
+                // Needed for vertical alignment to work
+                structure += " height=\"100%\"";
+            }
+            structure += "><tbody>"
                     + (orientationMode == ORIENTATION_HORIZONTAL ? "<tr valign=\"top\"></tr>"
                             : "") + "</tbody></table>";
+
             DOM.setInnerHTML(getElement(), structure);
             root = DOM.getFirstChild(getElement());
             // set TBODY to be the wrappedChildContainer