From: Leif Åstrand <leif@vaadin.com>
Date: Thu, 15 Mar 2012 08:22:29 +0000 (+0200)
Subject: Don't determine row height before the contents are rendered (#8313)
X-Git-Tag: 7.0.0.alpha2~309^2~2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=8ca6ae2a5ea772d8dbac11c0a6915e1d840e5b40;p=vaadin-framework.git

Don't determine row height before the contents are rendered (#8313)
---

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
index d2e4339ae0..a5d7d9b5b3 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
@@ -1608,6 +1608,9 @@ public class VScrollTable extends FlowPanel implements HasWidgets,
             return;
         }
         sizeNeedsInit = false;
+
+        scrollBody.setContainerHeight();
+
         /*
          * We will use browsers table rendering algorithm to find proper column
          * widths. If content and header take less space than available, we will
@@ -4146,12 +4149,6 @@ public class VScrollTable extends FlowPanel implements HasWidgets,
             throw new UnsupportedOperationException();
         }
 
-        @Override
-        protected void onAttach() {
-            super.onAttach();
-            setContainerHeight();
-        }
-
         /**
          * Fix container blocks height according to totalRows to avoid
          * "bouncing" when scrolling