#3710 cleanup and comments in VForm layout

svn changeset:20753/svn branch:6.7

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VForm.java b/src/com/vaadin/terminal/gwt/client/ui/VForm.java
index 2bea7e3ffe94a2874d760c281fa72ce400d58d54..7b0106fc8352232e422d83ec3110d978f433a488 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VForm.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VForm.java
@@ -133,8 +133,6 @@ public class VForm extends ComplexPanel implements Container, KeyDownHandler {
         }\r
 \r
         updateSize();\r
-        // TODO Check if this is needed\r
-        client.runDescendentsLayout(this);\r
 \r
         // first render footer so it will be easier to handle relative height of\r
         // main layout\r
@@ -153,11 +151,13 @@ public class VForm extends ComplexPanel implements Container, KeyDownHandler {
             }\r
             footer = newFooter;\r
             footer.updateFromUIDL(uidl.getChildUIDL(1), client);\r
+            // needed for the main layout to know the space it has available\r
             updateSize();\r
         } else {\r
             if (footer != null) {\r
                 remove((Widget) footer);\r
                 client.unregisterPaintable(footer);\r
+                // needed for the main layout to know the space it has available\r
                 updateSize();\r
             }\r
         }\r
@@ -175,7 +175,8 @@ public class VForm extends ComplexPanel implements Container, KeyDownHandler {
         }\r
         lo.updateFromUIDL(layoutUidl, client);\r
 \r
-        // recalculate size the footer for forms of undefined size - see #3710\r
+        // also recalculates size of the footer if undefined size form - see\r
+        // #3710\r
         updateSize();\r
         client.runDescendentsLayout(this);\r
 \r