From 66debbb208059d8bc2c6f97388b30763680e30f9 Mon Sep 17 00:00:00 2001 From: Artur Signell Date: Mon, 9 May 2016 09:29:40 +0300 Subject: Clarify expand javadoc for GridLayout Change-Id: If68e81c7116c9294754167981ea3fb53360501bb --- server/src/main/java/com/vaadin/ui/GridLayout.java | 22 ++++++++++++++++------ 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/server/src/main/java/com/vaadin/ui/GridLayout.java b/server/src/main/java/com/vaadin/ui/GridLayout.java index 148fd85fff..3b9272b034 100644 --- a/server/src/main/java/com/vaadin/ui/GridLayout.java +++ b/server/src/main/java/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) * -- cgit v1.2.3