From: Artur Signell <artur.signell@itmill.com>
Date: Mon, 3 Nov 2008 13:49:24 +0000 (+0000)
Subject: Fixed Firefox flickering issue in SplitPanel.
X-Git-Tag: 6.7.0.beta1~3883
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=3e0f4b5ab9852221d6495da76cf8029206b303cc;p=vaadin-framework.git

Fixed Firefox flickering issue in SplitPanel.
Removed SplitPanel IE7 hacks which should be unnecessary.

svn changeset:5799/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ui/ISplitPanel.java b/src/com/itmill/toolkit/terminal/gwt/client/ui/ISplitPanel.java
index ce0726cc59..5ba5cc7645 100644
--- a/src/com/itmill/toolkit/terminal/gwt/client/ui/ISplitPanel.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ui/ISplitPanel.java
@@ -118,18 +118,6 @@ public class ISplitPanel extends ComplexPanel implements Container,
         DOM.setStyleAttribute(splitter, "position", "absolute");
         DOM.setStyleAttribute(secondContainer, "position", "absolute");
 
-        DOM.setStyleAttribute(firstContainer, "overflow", "auto");
-        DOM.setStyleAttribute(secondContainer, "overflow", "auto");
-        if (BrowserInfo.get().isIE7()) {
-            /*
-             * Part I of IE7 weirdness hack, will be set to auto in layout phase
-             * 
-             * With IE7 one will sometimes get scrollbars with overflow auto
-             * even though there is nothing to scroll (content fits into area).
-             */
-            DOM.setStyleAttribute(firstContainer, "overflow", "hidden");
-            DOM.setStyleAttribute(secondContainer, "overflow", "hidden");
-        }
     }
 
     private void setOrientation(int orientation) {
@@ -225,11 +213,6 @@ public class ISplitPanel extends ComplexPanel implements Container,
         int wholeSize;
         int pixelPosition;
 
-        if (!(resizing && BrowserInfo.get().isGecko())) {
-            DOM.setStyleAttribute(firstContainer, "overflow", "hidden");
-            DOM.setStyleAttribute(secondContainer, "overflow", "hidden");
-        }
-
         switch (orientation) {
         case ORIENTATION_HORIZONTAL:
             wholeSize = DOM.getElementPropertyInt(wrapper, "clientWidth");
@@ -300,22 +283,7 @@ public class ISplitPanel extends ComplexPanel implements Container,
             break;
         }
 
-        if (Util.isIE7()) {
-            // Part I of IE7 weirdness hack, will be set to auto in layout phase
-            client.runDescendentsLayout(this);
-            DeferredCommand.addCommand(new Command() {
-                public void execute() {
-                    DOM.setStyleAttribute(firstContainer, "overflow", "auto");
-                    DOM.setStyleAttribute(secondContainer, "overflow", "auto");
-                }
-            });
-        } else {
-            client.runDescendentsLayout(this);
-            if (!(resizing && BrowserInfo.get().isGecko())) {
-                DOM.setStyleAttribute(firstContainer, "overflow", "auto");
-                DOM.setStyleAttribute(secondContainer, "overflow", "auto");
-            }
-        }
+        client.runDescendentsLayout(this);
 
         renderInformation.updateSize(getElement());