[lmi-commits] [lmi] master a59f47d 1/2: Slightly simplify keystrokes in a GUI test

[Greg Chicares]

branch: master
commit a59f47d505e9f20fe0b213d4df29c3196316b8ef
Author: Gregory W. Chicares <gchicares@sbcglobal.net>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>

    Slightly simplify keystrokes in a GUI test
    
    For experienced spreadsheet users who favor the keyboard, Esc is the
    natural way to clear a selection.
    
    The purpose of Ctrl-Home here is to move focus to the top row. It also
    happens to move to the leftmost column, which doesn't matter: Ctrl-Up
    would be as good for this purpose. (Users from the "123" era might favor
    End-Up, which works in gnumeric as well as the dominant msw spreadsheet,
    but not in libreoffice calc or wxGrid.)
    
    It is not necessary to highlight the top row; Ctrl-Home or Ctrl-Up makes
    it the "active" row.
---
 wx_test_validate_output.cpp | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/wx_test_validate_output.cpp b/wx_test_validate_output.cpp
index c527848..8399085 100644
--- a/wx_test_validate_output.cpp
+++ b/wx_test_validate_output.cpp
@@ -344,9 +344,8 @@ void validate_run_cell_and_copy_output
 
     wxUIActionSimulator ui;
 
-    ui.Char(WXK_UP);                  // Clear the current selection if any.
-    ui.Char(WXK_HOME, wxMOD_CONTROL); // Go to the left top cell.
-    ui.Char(WXK_RIGHT, wxMOD_SHIFT);  // Select the first row.
+    ui.Char(WXK_ESCAPE);              // Clear any selection.
+    ui.Char(WXK_HOME, wxMOD_CONTROL); // Move focus to top row.
 
     ui.Char('r', wxMOD_CONTROL);      // "Census|Run cell"
     wxYield();