diff options
| author | John Alhroos <john.ahlroos@itmill.com> | 2013-09-30 08:53:18 +0000 |
|---|---|---|
| committer | John Alhroos <john.ahlroos@itmill.com> | 2013-09-30 08:53:18 +0000 |
| commit | 1d645c4a18a4e24a80ad0351a90ab0f177c7e84f (patch) | |
| tree | 6e11df25b990dcef4390bbdb8b60402ec51cdd71 | |
| parent | 2c72763b3b34e9cb1e29d6946f75f300eed64eab (diff) | |
| download | vaadin-framework-1d645c4a18a4e24a80ad0351a90ab0f177c7e84f.tar.gz vaadin-framework-1d645c4a18a4e24a80ad0351a90ab0f177c7e84f.zip | |
Fixed regression where scrolling position changed unintentionally after #12407
svn changeset:26153/svn branch:6.8
| -rw-r--r-- | src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java | 8 |
1 files 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() { |