diff options
author | John Ahlroos <john@vaadin.com> | 2013-10-07 12:37:50 +0300 |
---|---|---|
committer | John Ahlroos <john@vaadin.com> | 2013-10-07 12:37:50 +0300 |
commit | d9f6dadfb41c2b618390edf8a05bbe78a8794e06 (patch) | |
tree | 89233bcea69462e20a7c9f1946f276d83e95c274 | |
parent | 503e5757bbac1a28c8987ecae9f75d271a61899b (diff) | |
download | vaadin-framework-d9f6dadfb41c2b618390edf8a05bbe78a8794e06.tar.gz vaadin-framework-d9f6dadfb41c2b618390edf8a05bbe78a8794e06.zip |
Fixed Table range selection IE regression #12407
After fixes for #12407 the range selection did not work in the case
where the selection start had previously been removed. This caused
MultiSelectWithRemovedRow test to fail on IE.
Change-Id: Iaa91cc6a3a310aedc80c4c2475fa8e94a30a5563
-rw-r--r-- | client/src/com/vaadin/client/ui/VScrollTable.java | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/client/src/com/vaadin/client/ui/VScrollTable.java b/client/src/com/vaadin/client/ui/VScrollTable.java index 6e9e28ff25..9cec59a5a2 100644 --- a/client/src/com/vaadin/client/ui/VScrollTable.java +++ b/client/src/com/vaadin/client/ui/VScrollTable.java @@ -6110,7 +6110,13 @@ public class VScrollTable extends FlowPanel implements HasWidgets, .next(); setRowFocus(endRow); } + } else if (!startRow.isSelected()) { + // The start row is no longer selected (probably removed) + // and so we select from above + startRow = (VScrollTableRow) scrollBody.iterator().next(); + setRowFocus(endRow); } + // Deselect previous items if so desired if (deselectPrevious) { deselectAll(); |