From 32d36df538dcc8382174420c0c1a765cc23e923a Mon Sep 17 00:00:00 2001
From: Marc Englund <marc.englund@itmill.com>
Date: Fri, 22 Aug 2008 07:51:09 +0000
Subject: [PATCH] setPageLength() no longer resets scroll position (fixes
 #1952), setCurrentPageFirst*() resets page buffer.

svn changeset:5243/svn branch:trunk
---
 src/com/itmill/toolkit/ui/Table.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/com/itmill/toolkit/ui/Table.java b/src/com/itmill/toolkit/ui/Table.java
index 5ffebbc630..37a64bd089 100644
--- a/src/com/itmill/toolkit/ui/Table.java
+++ b/src/com/itmill/toolkit/ui/Table.java
@@ -678,8 +678,6 @@ public class Table extends AbstractSelect implements Action.Container,
     public void setPageLength(int pageLength) {
         if (pageLength >= 0 && this.pageLength != pageLength) {
             this.pageLength = pageLength;
-            // "scroll" to first row
-            setCurrentPageFirstItemIndex(0);
             // Assures the visual refresh
             resetPageBuffer();
             refreshRenderedCells();
@@ -747,6 +745,7 @@ public class Table extends AbstractSelect implements Action.Container,
         }
 
         // Assures the visual refresh
+        resetPageBuffer();
         refreshRenderedCells();
 
     }
@@ -1077,6 +1076,9 @@ public class Table extends AbstractSelect implements Action.Container,
                 newIndex = currentPageFirstItemIndex = size() - 1;
             }
         }
+        // Assures the visual refresh
+        resetPageBuffer();
+        refreshRenderedCells();
     }
 
     /**
-- 
2.39.5