From 1e8ad071350bc32d53f035379973aead50b75209 Mon Sep 17 00:00:00 2001 From: Pekka Hyvönen Date: Tue, 14 Mar 2017 15:06:07 +0200 Subject: Fix removing rows from the middle of Escalator (#8826) * Fix removing rows from the middle of Escalator Fixes #8825 --- client/src/main/java/com/vaadin/client/widgets/Escalator.java | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) (limited to 'client') diff --git a/client/src/main/java/com/vaadin/client/widgets/Escalator.java b/client/src/main/java/com/vaadin/client/widgets/Escalator.java index 7eb99badf7..5cfad18540 100644 --- a/client/src/main/java/com/vaadin/client/widgets/Escalator.java +++ b/client/src/main/java/com/vaadin/client/widgets/Escalator.java @@ -3130,14 +3130,8 @@ public class Escalator extends Widget y += spacerContainer.getSpacerHeight(i); } - /* - * this is how many rows appeared into the viewport from - * below - */ - final int rowsToUpdateDataOn = numberOfRows - - escalatorRowsToRemove; - final int start = Math.max(0, - escalatorRowCount - rowsToUpdateDataOn); + // #8825 update data starting from the first moved row + final int start = dirtyRowsStart; final int end = escalatorRowCount; for (int i = start; i < end; i++) { final TableRowElement tr = visualRowOrder.get(i); -- cgit v1.2.3