@bobfrankston/rmfmail 1.1.102 → 1.1.103

package/client/app.js

@@ -3044,20 +3044,26 @@ 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.
+    // 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
     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 && listSelectionCount <= 1)
             return;
         e.preventDefault();
         deleteSelection();