@bobfrankston/rmfmail 1.2.251 → 1.2.253

package/client/app.js

@@ -4620,6 +4620,22 @@ function closeToolbarDropdowns() {
             hideDropdownHard(dd);
     }
 }
+// Clicking in the MESSAGE PREVIEW left the menu open. The preview body is an
+// iframe, so its pointerdown never reaches this document's capture-phase
+// listener above — and the transparent shield that solves this is only built
+// when a compose/popout overlay exists, which the plain three-pane view has
+// none of. Half the window is that iframe, so "click outside the settings
+// menu but it does not go away" was the common case (Bob 2026-08-11).
+// The iframe already posts `iframePointerDown` for the context menu's benefit;
+// listen to the same signal instead of shielding the whole viewport (a shield
+// would eat the click that should also select a row).
+window.addEventListener("message", (e) => {
+    if (e.data?.type !== "iframePointerDown")
+        return;
+    closeToolbarDropdowns();
+    document.getElementById("rail-menu-backdrop")?.remove();
+    refreshToolbarOverlayShield();
+});
 function refreshToolbarOverlayShield() {
     const dropdownOpen = ["settings-dropdown", "view-dropdown", "restart-dropdown"]
         .some(id => {
@@ -6494,6 +6510,32 @@ document.addEventListener("mailx-popout-message", (async (e) => {
             draftFolderId: msg.folderId,
             draftId: msg.mailxDraftId || "",
         };
+        // Reopening a draft must bring its attachments back into compose,
+        // exactly as forward / edit-as-new do. Drafts had none to bring until
+        // saveDraft started including them (2026-08-11); without this, editing
+        // a draft and re-saving would strip the files a second time.
+        if (Array.isArray(msg.attachments) && msg.attachments.length) {
+            const loaded = [];
+            for (const att of msg.attachments) {
+                try {
+                    const data = await getAttachment(accountId, msg.uid, att.id, msg.folderId);
+                    if (data?.content) {
+                        loaded.push({
+                            filename: data.filename || att.filename || "attachment",
+                            mimeType: data.contentType || att.contentType || "application/octet-stream",
+                            dataBase64: data.content,
+                        });
+                    }
+                }
+                catch (e) {
+                    // Loud: a dropped attachment here means the next autosave
+                    // writes a draft without it.
+                    console.error(`[compose] edit draft: failed to load attachment ${att.id} ("${att.filename}"): ${e?.message || e}`);
+                }
+            }
+            if (loaded.length)
+                init.attachments = loaded;
+        }
         sessionStorage.setItem("composeInit", JSON.stringify(init));
         showComposeOverlay(msg.subject ? `Edit: ${msg.subject}` : "Edit draft");
         return;