diff options
Diffstat (limited to 'src/com/vaadin/ui/AbsoluteLayout.java')
-rw-r--r-- | src/com/vaadin/ui/AbsoluteLayout.java | 19 |
1 files changed, 17 insertions, 2 deletions
diff --git a/src/com/vaadin/ui/AbsoluteLayout.java b/src/com/vaadin/ui/AbsoluteLayout.java index 3b0239af7b..c7a1583d28 100644 --- a/src/com/vaadin/ui/AbsoluteLayout.java +++ b/src/com/vaadin/ui/AbsoluteLayout.java @@ -122,8 +122,23 @@ public class AbsoluteLayout extends AbstractLayout implements * The css position string */ public void addComponent(Component c, String cssPosition) { - addComponent(c); - getPosition(c).setCSSString(cssPosition); + /* + * Create position instance and add it to componentToCoordinates map. We + * need to do this before we call addComponent so the attachListeners + * can access this position. #6368 + */ + ComponentPosition position = new ComponentPosition(); + position.setCSSString(cssPosition); + componentToCoordinates.put(c, position); + + try { + addComponent(c); + + } catch (IllegalArgumentException e) { + // Remove component coordinates if adding fails + componentToCoordinates.remove(c); + throw e; + } } /** |