From: Marc Englund <marc.englund@itmill.com>
Date: Fri, 1 Feb 2008 12:23:13 +0000 (+0000)
Subject: changed order of component adding, in case super.addComponent() throws...
X-Git-Tag: 6.7.0.beta1~5113
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=3cce6ee15bf75db2a6ba90934100a25bcca040ff;p=vaadin-framework.git

changed order of component adding, in case super.addComponent() throws...

svn changeset:3703/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/ui/OrderedLayout.java b/src/com/itmill/toolkit/ui/OrderedLayout.java
index e720caf619..6eefd72a46 100644
--- a/src/com/itmill/toolkit/ui/OrderedLayout.java
+++ b/src/com/itmill/toolkit/ui/OrderedLayout.java
@@ -128,8 +128,8 @@ public class OrderedLayout extends AbstractLayout {
      *                the component to be added.
      */
     public void addComponent(Component c) {
-        components.add(c);
         super.addComponent(c);
+        components.add(c);
         requestRepaint();
     }
 
@@ -141,8 +141,8 @@ public class OrderedLayout extends AbstractLayout {
      *                the component to be added.
      */
     public void addComponentAsFirst(Component c) {
-        components.addFirst(c);
         super.addComponent(c);
+        components.addFirst(c);
         requestRepaint();
     }
 
@@ -156,8 +156,8 @@ public class OrderedLayout extends AbstractLayout {
      *                currently in and after the position are shifted forwards.
      */
     public void addComponent(Component c, int index) {
-        components.add(index, c);
         super.addComponent(c);
+        components.add(index, c);
         requestRepaint();
     }