From: Matti Tahvonen Date: Mon, 21 Dec 2009 16:22:35 +0000 (+0000) Subject: svn changeset:10500/svn branch:6.2 X-Git-Tag: 6.7.0.beta1~2084^2~33 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=b6e1c57f743fc825de8333c07804ef8011f426b6;p=vaadin-framework.git svn changeset:10500/svn branch:6.2 --- diff --git a/src/com/vaadin/ui/GridLayout.java b/src/com/vaadin/ui/GridLayout.java index f1195e9ef4..19abe8d142 100644 --- a/src/com/vaadin/ui/GridLayout.java +++ b/src/com/vaadin/ui/GridLayout.java @@ -830,8 +830,9 @@ public class GridLayout extends AbstractLayout implements } /** - * An Exception object which is thrown when two Items occupy - * the same space on a grid. + * Gridlayout does not support laying components on top of each other. An + * OverlapsException is thrown when a component already exists + * (even partly) at the same space on a grid with the new component. * * @author IT Mill Ltd. * @version @@ -851,6 +852,32 @@ public class GridLayout extends AbstractLayout implements this.existingArea = existingArea; } + @Override + public String getMessage() { + StringBuilder sb = new StringBuilder(); + Component component = existingArea.getComponent(); + sb.append(component); + sb.append("( type = "); + sb.append(component.getClass().getName()); + if (component.getCaption() != null) { + sb.append(", caption = \""); + sb.append(component.getCaption()); + sb.append("\""); + } + sb.append(")"); + sb.append(" is already added to "); + sb.append(existingArea.column1); + sb.append(","); + sb.append(existingArea.column1); + sb.append(","); + sb.append(existingArea.row1); + sb.append(","); + sb.append(existingArea.row2); + sb.append("(column1, column2, row1, row2)."); + + return sb.toString(); + } + /** * Gets the area . *