@bobfrankston/rmfmail 1.1.103 → 1.1.104

package/client/app.js

@@ -3044,26 +3044,19 @@ document.addEventListener("keydown", (e) => {
         }
     }
     // Ctrl+D or Delete = Delete selected messages.
-    // P15: don't hijack Delete inside text inputs / textareas / contenteditable.
-    // Single-selection-while-typing was the bug Bob reported 2026-05-20:
-    // search input has focus, list has an auto-selected row, user presses
-    // Delete to backspace a character → mailx hard-deletes the row. Data loss.
-    //
-    // The 2026-05-09 multi-select carve-out (Delete-key works on a list
-    // selection even when focus drifted into search) is preserved only for
-    // *explicit* multi-select (>1 row). Single selection is ALWAYS the
-    // auto-selected first row — never an explicit "delete this" intent when
-    // you're typing in search. So:
-    //   - focus in editable + ≤1 selected → native text edit (delete char)
-    //   - focus in editable + >1 selected → user multi-selected on purpose, run delete
-    //   - focus elsewhere → run delete
+    // Focus is the only signal. Text input has focus → Delete is a native
+    // character edit, ALWAYS. List has focus (a row, the list container,
+    // anything outside a text editor) → Delete acts on the list selection,
+    // single or multi. The 2026-05-09 "multi-select while typing" carve-out
+    // is gone — it cost Bob real messages 2026-05-20 by hard-deleting the
+    // auto-selected row every time he hit Delete to backspace in the search
+    // input.
     if ((e.ctrlKey && e.key === "d") || e.key === "Delete") {
         const t = e.target;
         const tag = t?.tagName;
         const editable = t?.isContentEditable;
         const inEditable = tag === "INPUT" || tag === "TEXTAREA" || tag === "SELECT" || editable;
-        const listSelectionCount = document.querySelectorAll("#ml-body .ml-row.selected").length;
-        if (inEditable && listSelectionCount <= 1)
+        if (inEditable)
             return;
         e.preventDefault();
         deleteSelection();
@@ -3084,10 +3077,18 @@ document.addEventListener("keydown", (e) => {
         e.preventDefault();
         document.getElementById("btn-sync")?.click();
     }
-    // R = Toggle read/unread
-    if (e.key.toLowerCase() === "r" && !e.ctrlKey && !e.metaKey && !e.altKey) {
+    // R (no modifiers) or Ctrl+Q = Toggle read/unread on the focused row.
+    // Ctrl+Q mirrors the Outlook-style "mark read/unread" combo so it works
+    // even when focus is in a text input (Ctrl-modifier guarantees no
+    // collision with typing). Bare R defers when typing.
+    const isToggleSeen = (e.key.toLowerCase() === "r" && !e.ctrlKey && !e.metaKey && !e.altKey) ||
+        (e.ctrlKey && !e.metaKey && !e.altKey && e.key.toLowerCase() === "q");
+    if (isToggleSeen) {
         const active = document.activeElement;
-        if (active && (active.tagName === "INPUT" || active.tagName === "TEXTAREA" || active.tagName === "SELECT"))
+        const inText = active && (active.tagName === "INPUT" || active.tagName === "TEXTAREA" || active.tagName === "SELECT" || active.isContentEditable);
+        // Bare R yields to text inputs; Ctrl+Q overrides them so it's reachable
+        // from the search box or compose draft list.
+        if (!e.ctrlKey && inText)
             return;
         const sel = getCurrentFocused();
         if (!sel)