From cb03f6b405f32170014a1d5b5fdff539d2b5fc42 Mon Sep 17 00:00:00 2001
From: John Alhroos <john.ahlroos@itmill.com>
Date: Wed, 5 May 2010 08:07:21 +0000
Subject: [PATCH] Fixes deselection when deselecting whole range outside of
 selection page.

svn changeset:13048/svn branch:6.4
---
 src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java     | 1 +
 tests/src/com/vaadin/tests/components/table/KeyControl.java | 4 +++-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
index de47f5197f..faaa182361 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
@@ -4067,6 +4067,7 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
         }
         // still ensure all selects are removed from (not necessary rendered)
         selectedRowKeys.clear();
+        selectedRowRanges.clear();
     }
 
     /**
diff --git a/tests/src/com/vaadin/tests/components/table/KeyControl.java b/tests/src/com/vaadin/tests/components/table/KeyControl.java
index a0a13e4858..b13c53a43f 100644
--- a/tests/src/com/vaadin/tests/components/table/KeyControl.java
+++ b/tests/src/com/vaadin/tests/components/table/KeyControl.java
@@ -89,10 +89,12 @@ public class KeyControl extends TestBase {
             public void valueChange(ValueChangeEvent event) {
                 Set<String> value = (Set<String>) table3.getValue();
                 selected3.setValue(value.size() == 0 ? "No selected items"
-                        : value);
+                        : value + ": Total " + value.size() + " items");
             }
         });
 
+        selected3.setWidth("300px");
+        selected3.setHeight("500px");
         layout3.addComponent(selected3);
         layout.addComponent(layout3);
     }
-- 
2.39.5