From 8ca6ae2a5ea772d8dbac11c0a6915e1d840e5b40 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Leif=20=C3=85strand?= <leif@vaadin.com>
Date: Thu, 15 Mar 2012 10:22:29 +0200
Subject: [PATCH] Don't determine row height before the contents are rendered
 (#8313)

---
 src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

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
-- 
2.39.5