From 1d645c4a18a4e24a80ad0351a90ab0f177c7e84f Mon Sep 17 00:00:00 2001 From: John Alhroos Date: Mon, 30 Sep 2013 08:53:18 +0000 Subject: Fixed regression where scrolling position changed unintentionally after #12407 svn changeset:26153/svn branch:6.8 --- src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java index e257513975..7dfdf2de8a 100644 --- a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java +++ b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java @@ -1259,8 +1259,12 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler, .getIntVariable("firstvisible") : 0; firstVisibleOnLastPage = uidl.hasVariable("firstvisibleonlastpage") ? uidl .getIntVariable("firstvisibleonlastpage") : -1; - - scrollToFirstVisible(); + if (firstvisible != lastRequestedFirstvisible) { + // Only scroll if the first visible changes from the server side. + // Else we might unintentionally scroll even when the scroll + // position has not changed. + scrollToFirstVisible(); + } } private void scrollToFirstVisible() { -- cgit v1.2.3