From f1665517f41d8e304e1fb864e8f9641074430ea7 Mon Sep 17 00:00:00 2001 From: Artur Signell Date: Mon, 9 May 2016 09:29:40 +0300 Subject: [PATCH] Clarify expand javadoc for GridLayout Change-Id: I0fd18d20f01ca497e5c434c913e564cfa6444494 --- server/src/com/vaadin/ui/GridLayout.java | 22 ++++++++++++++++------ 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/server/src/com/vaadin/ui/GridLayout.java b/server/src/com/vaadin/ui/GridLayout.java index 148fd85fff..3b9272b034 100644 --- a/server/src/com/vaadin/ui/GridLayout.java +++ b/server/src/com/vaadin/ui/GridLayout.java @@ -1082,9 +1082,14 @@ public class GridLayout extends AbstractLayout implements *

* *

- * Note that the component width of the GridLayout must be defined (fixed or - * relative, as opposed to undefined) for this method to have any effect. - *

+ * Note, that width of this GridLayout needs to be defined (fixed or + * relative, as opposed to undefined height) for this method to have any + * effect. + *

+ * Note that checking for relative width for the child components is done on + * the server so you cannot set a child component to have undefined width on + * the server and set it to 100% in CSS. You must set it to + * 100% on the server. * * @see #setWidth(float, int) * @@ -1120,9 +1125,14 @@ public class GridLayout extends AbstractLayout implements *

* *

- * Note, that height needs to be defined (fixed or relative, as opposed to - * undefined height) for this method to have any effect. - *

+ * Note, that height of this GridLayout needs to be defined (fixed or + * relative, as opposed to undefined height) for this method to have any + * effect. + *

+ * Note that checking for relative height for the child components is done + * on the server so you cannot set a child component to have undefined + * height on the server and set it to 100% in CSS. You must set + * it to 100% on the server. * * @see #setHeight(float, int) * -- 2.39.5