@bobfrankston/rmfmail 1.2.11 → 1.2.13

package/client/app.js

@@ -1495,23 +1495,18 @@ async function deleteSelectedMessages() {
     // the prompt and the undo promise must change: permanent delete always
     // confirms (even a single message) and can't be undone (Bob 2026-06-12).
     const inTrash = currentFolderSpecialUse === "trash";
-    // SAFETY GATE.
-    //  - In Trash: any delete is permanent → always confirm, warn it's final.
-    //  - Elsewhere: a single delete is instant (quick triage); a BULK delete
-    //    confirms, because `Ctrl+A` selects every visible row and in All
-    //    Inboxes that's a scattered screenful — exactly what silently trashed
-    //    114 of Bob's messages on 2026-06-12.
+    // SAFETY GATE — only for the IRREVERSIBLE action.
+    //  - In Trash: delete is permanent → confirm (Bob explicitly wanted this).
+    //  - Elsewhere: move to Trash with NO prompt, single or bulk. It's
+    //    undoable (Ctrl+Z restores the whole batch) and matches Outlook /
+    //    Thunderbird, which don't nag on a move-to-trash. Bob 2026-06-15:
+    //    "stop giving me the trash warning message. It is annoying."
     const n = selected.length;
     if (inTrash) {
         if (!confirm(`Permanently delete ${n} message${n === 1 ? "" : "s"} from Trash?\n\nThis cannot be undone.`)) {
             return;
         }
     }
-    else if (n > 1) {
-        if (!confirm(`Move ${n} messages to Trash?\n\n(Ctrl+Z restores them if this was a mistake.)`)) {
-            return;
-        }
-    }
     const statusSync = document.getElementById("status-sync");
     // Optimistic UI: remove from list IMMEDIATELY, then queue the IPC.
     // Old order awaited the daemon round-trip (IPC + DB updates) before