From 512aa1851012bfcf6e2758cf169d88655b99e429 Mon Sep 17 00:00:00 2001 From: Artur Signell Date: Tue, 23 Feb 2010 12:54:23 +0000 Subject: [PATCH] Fix for #4206 - Setting scroll position doesn't work with Webkit browsers svn changeset:11497/svn branch:6.3 --- src/com/vaadin/terminal/gwt/client/ui/VView.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/com/vaadin/terminal/gwt/client/ui/VView.java b/src/com/vaadin/terminal/gwt/client/ui/VView.java index 838b60a6d8..033d35dab1 100644 --- a/src/com/vaadin/terminal/gwt/client/ui/VView.java +++ b/src/com/vaadin/terminal/gwt/client/ui/VView.java @@ -372,10 +372,6 @@ public class VView extends SimplePanel implements Container, ResizeHandler, onResize(Window.getClientWidth(), Window.getClientHeight()); - if (BrowserInfo.get().isSafari()) { - Util.runWebkitOverflowAutoFix(getElement()); - } - // finally set scroll position from UIDL if (uidl.hasVariable("scrollTop")) { scrollable = true; @@ -387,6 +383,12 @@ public class VView extends SimplePanel implements Container, ResizeHandler, scrollable = false; } + // Safari workaround must be run after scrollTop is updated as it sets + // scrollTop using a deferred command. + if (BrowserInfo.get().isSafari()) { + Util.runWebkitOverflowAutoFix(getElement()); + } + rendering = false; } -- 2.39.5