Fixed #58: fixed shift-select to extend selection without needing mou…

…se movement
robmcmullen · Jun 4, 2016 · a6a3b8f · a6a3b8f
1 parent 5a2e877
commit a6a3b8f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/omnivore/utils/wx/bitviewscroller.py b/omnivore/utils/wx/bitviewscroller.py
@@ -391,8 +391,9 @@ def on_left_down(self, evt):
                 e.cursor_index = index1
             elif index2 > e.anchor_start_index:
                 e.anchor_end_index = index2
-                e.cursor_index = index2
+                e.cursor_index = index2 - 1
             e.anchor_initial_start_index, e.anchor_initial_end_index = e.anchor_start_index, e.anchor_end_index
+            e.select_range(e.anchor_start_index, e.anchor_end_index, add=self.multi_select_mode)
         else:
             e.anchor_initial_start_index, e.anchor_initial_end_index = index1, index2
             e.cursor_index = index1

diff --git a/omnivore/utils/wx/bytegrid.py b/omnivore/utils/wx/bytegrid.py
@@ -686,8 +686,9 @@ def on_left_down(self, evt):
                 e.cursor_index = index1
             elif index2 > e.anchor_start_index:
                 e.anchor_end_index = index2
-                e.cursor_index = index2
+                e.cursor_index = index2 - 1
             e.anchor_initial_start_index, e.anchor_initial_end_index = e.anchor_start_index, e.anchor_end_index
+            e.select_range(e.anchor_start_index, e.anchor_end_index, add=self.multi_select_mode)
         else:
             self.ClearSelection()
             index1, index2 = self.table.get_index_range(r, c)