@bobfrankston/rmfmail 1.1.102 → 1.1.104

package/client/app.js

@@ -3044,20 +3044,19 @@ document.addEventListener("keydown", (e) => {
         }
     }
     // Ctrl+D or Delete = Delete selected messages.
-    // P15: don't hijack Delete inside text inputs / textareas / contenteditable
-    // — JSONC editor's Delete key was being eaten because we always preventDefault'd.
-    // 2026-05-09 fix: if the LIST has multiple selected rows, the user
-    // clearly means "delete those" regardless of where focus is — search
-    // input retaining focus after filtering shouldn't make the key dead.
-    // Only defer to native behavior when there's no list selection AND
-    // focus is in a text-editing surface.
+    // 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 === 0)
+        if (inEditable)
             return;
         e.preventDefault();
         deleteSelection();
@@ -3078,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)