From: Matti Tahvonen <matti.tahvonen@itmill.com>
Date: Mon, 22 Sep 2008 09:59:54 +0000 (+0000)
Subject: added FIXME note
X-Git-Tag: 6.7.0.beta1~4123
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=cdd0903578a914b9a86e6f538962ee4e631a6c09;p=vaadin-framework.git

added FIXME note

svn changeset:5477/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ui/IOrderedLayout.java b/src/com/itmill/toolkit/terminal/gwt/client/ui/IOrderedLayout.java
index 7dea7ec1da..48b01c51d5 100644
--- a/src/com/itmill/toolkit/terminal/gwt/client/ui/IOrderedLayout.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ui/IOrderedLayout.java
@@ -814,6 +814,9 @@ public class IOrderedLayout extends Panel implements Container,
         DOM.setStyleAttribute(measure, "height", "100%");
         Element parent = DOM.getParent(getElement());
         DOM.insertBefore(parent, measure, getElement());
+        // FIXME Do not detach from DOM this way. At least proper detach, attach
+        // must be called. Affects odd behavior in childs, performance issues
+        // and flickering. See #2102
         DOM.removeChild(parent, getElement());
         int size = DOM.getElementPropertyInt(measure, offset);
         DOM.insertBefore(parent, getElement(), measure);