@bobfrankston/rmfmail 1.2.192 → 1.2.193

package/client/app.js

@@ -1788,6 +1788,16 @@ async function performUndo() {
                 statusSync.textContent = msgs.length === 1
                     ? "Message restored"
                     : `Restored ${msgs.length} messages`;
+            // Position at what was just restored — an undo whose effect lands
+            // off-screen (or in a folder you've since left) reads as a no-op.
+            // Reveal the first of the batch; the rest are in the same view.
+            // On a miss revealMessage restores the prior view itself.
+            const first = msgs[0];
+            if (first?.folderId) {
+                await revealMessage(first.accountId, first.folderId, first.uid);
+                syncChromeToListView();
+                return;
+            }
         }
         else if (op.kind === "move") {
             const { messages } = op.payload;
@@ -1807,6 +1817,14 @@ async function performUndo() {
             }
             if (statusSync)
                 statusSync.textContent = `Undid move of ${messages.length} message${messages.length !== 1 ? "s" : ""}`;
+            // Same positioning rule as undo-delete: show the message back in
+            // its original folder.
+            const firstMoved = messages[0];
+            if (firstMoved?.sourceFolderId) {
+                await revealMessage(firstMoved.accountId, firstMoved.sourceFolderId, firstMoved.uid);
+                syncChromeToListView();
+                return;
+            }
         }
         else if (op.kind === "flag") {
             const { updateFlags } = await import("./lib/api-client.js");
@@ -2759,6 +2777,13 @@ async function closeSearchAndReveal() {
     // claimed search-on-All-Inboxes and the list looked broken (Bob
     // 2026-07-04 "cleared the search but it isn't restoring the list").
     // Sync tab, tree highlight and titles to what the list actually shows.
+    syncChromeToListView();
+}
+/** Mirror tab, folder-tree highlight and titles to whatever view the message
+ *  list is actually showing. Needed after any list-driven navigation
+ *  (revealMessage from search-close or undo) — the list module can land on a
+ *  different folder than the surrounding chrome still claims. */
+function syncChromeToListView() {
     const vw = getCurrentView();
     if (vw.unified) {
         highlightFolder(null, -1);