@bobfrankston/rmfmail 1.2.221 → 1.2.222

package/client/app.ts

@@ -1449,6 +1449,27 @@ function showComposeOverlay(title = "Compose"): HTMLIFrameElement {
     return frame;
 }
 
+// Native context-menu commands land in the TOP frame: msger dispatches them
+// with CoreWebView2.ExecuteScript, which always runs against the top-level
+// document. But the in-window compose is an IFRAME, and the handler that owns
+// the semantics (__msgerContextCommand) is defined inside it — so every
+// right-click item in the in-window compose hit msger's `window.__msgerContext
+// Command && …` guard, found nothing, and silently did nothing. The items were
+// registered for this window in v1.2.155; the dispatcher never was. Forward to
+// the frontmost compose overlay (same origin, so reaching into it is legal).
+(window as any).__msgerContextCommand = (id: string): void => {
+    const frames = [...document.querySelectorAll<HTMLIFrameElement>(".compose-overlay iframe")];
+    // Frontmost = highest z-index on the wrapper; mousedown maintains it, so
+    // the one the user just right-clicked is the one on top.
+    const target = frames.sort((a, b) => {
+        const z = (el: HTMLIFrameElement) => Number((el.closest(".compose-overlay") as HTMLElement)?.style.zIndex || 0);
+        return z(b) - z(a);
+    })[0];
+    try {
+        (target?.contentWindow as any)?.__msgerContextCommand?.(id);
+    } catch { /* frame torn down mid-click */ }
+};
+
 /** Drop a transparent full-viewport shield in front of every other element
  *  so mousemove events stay in the document during a drag. Setting
  *  pointer-events:none on the compose iframe alone wasn't enough — the

package/client/compose/compose.bundle.js

@@ -2080,7 +2080,13 @@ async function createTinyMceEditor(container2, opts = {}) {
       // 2026-06-01). The paste_* OPTIONS below remain valid (core).
       plugins: "lists advlist link table code codesample image searchreplace autolink wordcount emoticons charmap insertdatetime quickbars nonbreaking directionality help",
       toolbar: [
-        "undo redo | bold italic underline strikethrough | forecolor backcolor",
+        // fontsize on the main bar (Bob 2026-08-07: "there should also
+        // be font sizing on the tool bar"). `fontsize` is the v6+ name
+        // for v5's `fontsizeselect`; it renders the size dropdown and
+        // reflects the size at the caret. The Format ▸ Font sizes
+        // submenu had it all along, which is two clicks deeper than a
+        // size control has any business being.
+        "undo redo | fontsize | bold italic underline strikethrough | forecolor backcolor",
         // blockquote on the main bar (was quickbar-only) — Bob
         // 2026-07-28: pasted/inserted text needs a one-click quote.
         "blockquote bullist numlist outdent indent | link table image code rmfcode | emoticons charmap | help"
@@ -6094,6 +6100,60 @@ async function popoutHandoff() {
 window.addEventListener("compose-popout", () => {
   void popoutHandoff();
 });
+async function runClipboardCommand(id) {
+  const ne = editor?.nativeEditor;
+  const isTiny = !!ne && typeof ne.execCommand === "function" && !!ne.selection;
+  const label = id === "cut" ? "Cut" : id === "copy" ? "Copy" : "Paste";
+  const shortcut = id === "cut" ? "Ctrl+X" : id === "copy" ? "Ctrl+C" : "Ctrl+V";
+  try {
+    if (id === "paste") {
+      let html2 = "";
+      let text2 = "";
+      if (navigator.clipboard?.read) {
+        for (const item of await navigator.clipboard.read()) {
+          if (item.types.includes("text/html")) html2 = await (await item.getType("text/html")).text();
+          if (item.types.includes("text/plain")) text2 = await (await item.getType("text/plain")).text();
+        }
+      } else {
+        text2 = await navigator.clipboard.readText();
+      }
+      const content = html2 || (text2 ? text2.replace(/[&<>]/g, (c) => ({ "&": "&amp;", "<": "&lt;", ">": "&gt;" })[c]).replace(/\r?\n/g, "<br>") : "");
+      if (!content) return;
+      if (isTiny) ne.execCommand("mceInsertContent", false, content);
+      else if (ne && typeof ne.getSelection === "function") {
+        const sel = ne.getSelection(true);
+        ne.clipboard?.dangerouslyPasteHTML?.(sel?.index ?? 0, content);
+      } else throw new Error("no editor");
+      return;
+    }
+    if (isTiny) {
+      try {
+        if (ne.selection.isCollapsed() && typeof ne.selection.expand === "function") ne.selection.expand({ type: "word" });
+      } catch {
+      }
+    }
+    const html = isTiny ? ne.selection.getContent({ format: "html" }) : "";
+    const text = isTiny ? ne.selection.getContent({ format: "text" }) : window.getSelection()?.toString() || "";
+    if (!text && !html) return;
+    if (html && typeof ClipboardItem === "function" && navigator.clipboard?.write) {
+      await navigator.clipboard.write([new ClipboardItem({
+        "text/html": new Blob([html], { type: "text/html" }),
+        "text/plain": new Blob([text], { type: "text/plain" })
+      })]);
+    } else {
+      await navigator.clipboard.writeText(text);
+    }
+    if (id === "cut") {
+      if (isTiny) ne.execCommand("Delete");
+      else if (ne && typeof ne.deleteText === "function") {
+        const sel = ne.getSelection();
+        if (sel?.length) ne.deleteText(sel.index, sel.length);
+      }
+    }
+  } catch (e) {
+    showDraftStatus(`${label} failed: ${e?.message || e}. ${shortcut} still works.`, true);
+  }
+}
 window.__msgerContextCommand = (id) => {
   if (id === "source") {
     try {
@@ -6102,6 +6162,10 @@ window.__msgerContextCommand = (id) => {
     }
     return;
   }
+  if (id === "cut" || id === "copy" || id === "paste") {
+    void runClipboardCommand(id);
+    return;
+  }
   const ne = editor?.nativeEditor;
   if (!ne) return;
   const isTiny = typeof ne.execCommand === "function" && ne.selection;