From: Artur Signell <artur.signell@itmill.com>
Date: Tue, 16 Aug 2011 09:22:39 +0000 (+0000)
Subject: #7183 Table in single select mode fires only 1 change event on iPad
X-Git-Tag: 6.7.0.beta1~72
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=1027c03f4a5b2e6cfd9da81e7bcc94c5a7a4f8da;p=vaadin-framework.git

#7183 Table in single select mode fires only 1 change event on iPad

svn changeset:20408/svn branch:6.7
---

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
index b3f7f86d0e..32e238024e 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java
@@ -4756,7 +4756,19 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler,
                                                     .contains(getKey());
 
                                     if (!currentlyJustThisRowSelected) {
-                                        if (isMultiSelectModeDefault()) {
+                                        if (isSingleSelectMode()
+                                                || isMultiSelectModeSimple()) {
+                                            /*
+                                             * For default multi select mode
+                                             * (ctrl/shift) and for single
+                                             * select mode we need to clear the
+                                             * previous selection before
+                                             * selecting a new one when the user
+                                             * clicks on a row. Only in
+                                             * multiselect/simple mode the old
+                                             * selection should remain after a
+                                             * normal click.
+                                             */
                                             deselectAll();
                                         }
                                         toggleSelection();