feat(ui): make Escape deselect the row as well as blank the pane

Blanking the pane while leaving the row highlighted reads as half an
action, and deselecting is what Escape means nearly everywhere else. The
user asked for two actions rather than a changed one, so clear_pane keeps
its behaviour and moves to Shift+Esc; clear_selection takes Escape and
does both. Shift+Esc rather than unbound because every action carries a
default and a test enforces it.

Clearing the selection re-adopts the thread it just cleared, unless done
in exactly the right way. clearSelection() leaves currentIndex() valid,
so onSelectionChanged takes its "one or fewer rows" branch, sees a
current row whose id differs from m_currentThreadId, and calls
onThreadSelected for it. Clearing the selection before blanking lets that
run while the id still matches, so nothing reloads, and clearing current
stops a later collapse-to-one-row reaching the same row. All four
arrangements were tried; only this one passes.

The first version of the test could not distinguish any of them. It
asserted showingPlaceholder(), which passes regardless because this
fixture has no worker, so loadThread never replies and the pane is never
repainted. currentThreadId() and currentIndex() are observable without
one, and asserting those is what made the test discriminate.

1 files changed, 12 insertions, 1 deletions

diff --git a/src/keymap.cpp b/src/keymap.cpp
index a16a700..29a5f71 100644
--- a/src/keymap.cpp
+++ b/src/keymap.cpp
@@ -38,6 +38,7 @@ QStringList KeyMap::knownActions()
         QStringLiteral("complete_query"),
         QStringLiteral("select_all"),
         QStringLiteral("clear_pane"),
+        QStringLiteral("clear_selection"),
         QStringLiteral("toggle_html"),
         QStringLiteral("load_remote"),
         QStringLiteral("message_details"),
@@ -85,7 +86,17 @@ QList<QPair<QString, QString>> KeyMap::defaultBindings()
         // Escape is not claimed by anything else at window level. The query
         // completer handles its own Escape while its popup is up, and a popup
         // consumes the key before a window shortcut sees it.
-        { QStringLiteral("Esc"),          QStringLiteral("clear_pane") },
+        //
+        // It clears the SELECTION as well as the pane (item 50). Deselecting is
+        // what Escape means nearly everywhere else, and blanking a pane while
+        // leaving the row highlighted reads as half an action.
+        //
+        // clear_pane keeps the narrower behaviour on Shift+Esc: same key, and
+        // the modifier reads as "less than the plain one". It needs SOME
+        // default rather than being left unbound, since every action carries
+        // one and everyActionHasAShortcut enforces exactly that.
+        { QStringLiteral("Esc"),          QStringLiteral("clear_selection") },
+        { QStringLiteral("Shift+Esc"),    QStringLiteral("clear_pane") },
         { QStringLiteral("Ctrl+H"),       QStringLiteral("toggle_html") },
         { QStringLiteral("Ctrl+M"),       QStringLiteral("load_remote") },
         // Shifted because Ctrl+D is delete. Both are "D for details/delete"