@bobfrankston/rmfmail 1.2.81 → 1.2.83

package/client/app.bundle.js

@@ -2418,6 +2418,7 @@ function popOutToWindow() {
     return;
   const p = window.__rmfPopout;
   if (!p?.base || !p?.token) {
+    console.log("[popout-window] no server info (__rmfPopout missing) \u2192 overlay fallback");
     popOutCurrentMessage();
     return;
   }
@@ -2425,9 +2426,28 @@ function popOutToWindow() {
   const u = currentMessage.uid;
   const f = currentMessage.folderId;
   const url = `${p.base}/v?a=${encodeURIComponent(a)}&u=${u}&f=${f ?? ""}&t=${encodeURIComponent(p.token)}`;
-  const w = window.open(url, `rmfmsg-${a}-${u}`, "width=820,height=900,resizable=yes,scrollbars=yes");
-  if (!w)
+  console.log(`[popout-window] opening ${url}`);
+  let w = null;
+  try {
+    w = window.open(url, `rmfmsg-${a}-${u}`, "width=820,height=900,resizable=yes,scrollbars=yes");
+  } catch (e) {
+    console.log(`[popout-window] window.open THREW: ${e?.message || e} \u2192 overlay fallback`);
+    popOutCurrentMessage();
+    return;
+  }
+  if (!w) {
+    console.log("[popout-window] window.open returned NULL (host blocked it) \u2192 overlay fallback");
     popOutCurrentMessage();
+    return;
+  }
+  console.log("[popout-window] window.open returned a window object; verifying it stays open\u2026");
+  setTimeout(() => {
+    try {
+      console.log(`[popout-window] +600ms: window.closed=${w.closed}`);
+    } catch (e) {
+      console.log(`[popout-window] +600ms: cannot read .closed (${e?.message || e})`);
+    }
+  }, 600);
 }
 function printCurrentMessage() {
   if (!currentMessage)
@@ -2551,22 +2571,26 @@ function spawnDesktopPopout(msg, accountId) {
     wrapper.style.right = `${20 + existing * 28}px`;
   }
   const toolbar = document.createElement("div");
-  toolbar.style.cssText = "display:flex;gap:6px;padding:6px 12px;border-bottom:1px solid var(--color-border, #ddd);flex-shrink:0;";
-  const mkPopoutBtn = (label, title, onClick) => {
+  toolbar.className = "mv-toolbar";
+  toolbar.style.cssText = "display:flex;gap:4px;padding:6px 12px;border-bottom:1px solid var(--color-border, #ddd);flex-shrink:0;";
+  const mkPopoutBtn = (glyph, title, onClick) => {
     const b = document.createElement("button");
-    b.textContent = label;
+    b.className = "tb-btn";
+    b.textContent = glyph;
     b.title = title;
-    b.style.cssText = "padding:4px 10px;font-size:13px;cursor:pointer;border:1px solid var(--color-border, #ccc);border-radius:4px;background:var(--color-bg, #fff);color:var(--color-text, #000);";
+    b.style.cssText = "padding:4px 9px;font-size:15px;cursor:pointer;border:1px solid var(--color-border, #ccc);border-radius:4px;background:var(--color-bg, #fff);color:var(--color-text, #000);";
     b.addEventListener("click", onClick);
     return b;
   };
   const firePopoutAction = (action) => {
     document.dispatchEvent(new CustomEvent("mailx-popout-action", { detail: { action, msg, accountId } }));
   };
-  toolbar.appendChild(mkPopoutBtn("Reply", "Reply", () => firePopoutAction("reply")));
-  toolbar.appendChild(mkPopoutBtn("Reply All", "Reply to all", () => firePopoutAction("replyAll")));
-  toolbar.appendChild(mkPopoutBtn("Forward", "Forward", () => firePopoutAction("forward")));
-  toolbar.appendChild(mkPopoutBtn("Delete", "Delete this message", () => {
+  toolbar.appendChild(mkPopoutBtn("\u21A9", "Reply (Ctrl+R)", () => firePopoutAction("reply")));
+  toolbar.appendChild(mkPopoutBtn("\u21A9\u21A9", "Reply All (Ctrl+Shift+R)", () => firePopoutAction("replyAll")));
+  toolbar.appendChild(mkPopoutBtn("\u2192", "Forward", () => firePopoutAction("forward")));
+  toolbar.appendChild(mkPopoutBtn("\u29C9", "Edit as new message", () => firePopoutAction("editAsNew")));
+  toolbar.appendChild(mkPopoutBtn("\u2605", "Flag", () => firePopoutAction("flag")));
+  toolbar.appendChild(mkPopoutBtn("\u{1F5D1}", "Delete", () => {
     firePopoutAction("delete");
     wrapper.remove();
   }));
@@ -11572,10 +11596,15 @@ ${detail}`;
 document.addEventListener("mailx-popout-action", ((e) => {
   const { action, msg, accountId } = e.detail || {};
   if (!msg) return;
-  if (action === "reply" || action === "replyAll" || action === "forward") {
+  if (action === "reply" || action === "replyAll" || action === "forward" || action === "editAsNew") {
     openCompose(action, msg, accountId);
   } else if (action === "delete") {
     deleteMessage(accountId, msg.uid).catch((err) => console.error(`[popout] delete failed: ${err?.message || err}`));
+  } else if (action === "flag") {
+    const wasFlagged = flaggedOf(msg);
+    setFlagged(msg, !wasFlagged);
+    updateFlags(accountId, msg.uid, msg.flags, msg.folderId).catch((err) => console.error(`[popout] flag failed: ${err?.message || err}`));
+    setRowFlagged(accountId, msg.uid, !wasFlagged);
   }
 }));
 document.addEventListener("mailx-popout-message", (async (e) => {