@bobfrankston/rmfmail 1.2.274 → 1.2.277

package/client/app.js

@@ -159,6 +159,9 @@ window.__btick && window.__btick("app.ts module body executing");
 let baseTitle = APP_NAME;
 let lastSeenCount = 0;
 let badgeCount = 0;
+/** Last count pushed to the Windows taskbar overlay, so an unchanged count
+ *  is not re-pushed. -1 = nothing pushed yet. See updateBadge. */
+let lastOverlayCount = -1;
 function updateBadge(count) {
     badgeCount = count;
     // Update title
@@ -212,7 +215,19 @@ function updateBadge(count) {
         // taskbar for the whole life of this feature (Bob 2026-05-28 "are you
         // able to show the number of new messages").
         const hostApi = window.mailxapi;
-        if (hostApi?.setTaskbarOverlay) {
+        // Push ONLY when the number actually changes. Windows keeps the
+        // overlay icon alive until the next SetOverlayIcon, so the host
+        // cannot free the icon it just handed over — each push costs the
+        // host process one HICON plus the two bitmaps inside it, and the
+        // unread-count refresh fires every few seconds. That reached the
+        // 10,000-object per-process GDI cap in about a day, after which the
+        // host logged "CreateIconIndirect failed: Not enough memory
+        // resources" on every tick and could create no further GDI objects
+        // (Bob 2026-08-23, measured: rmfmail.exe at GDI=9995). msger now
+        // frees the PREVIOUS icon so the leak is bounded either way; this
+        // just stops the pointless churn (Claude Code 2026-08-23).
+        if (hostApi?.setTaskbarOverlay && count !== lastOverlayCount) {
+            lastOverlayCount = count;
             if (count > 0) {
                 // BADGE-ONLY overlay. SetOverlayIcon composites the supplied
                 // icon onto the bottom-right corner of the taskbar button —
@@ -6033,6 +6048,71 @@ document.getElementById("opt-compose-font-size")?.addEventListener("change", (e)
         saveSettings(settings);
     }).catch(() => { });
 });
+// TinyMCE UI-language picker. The options are whatever language packs
+// build-tinymce.js actually copied next to the script (langs/index.json), so
+// the menu can never offer a pack that would 404 at editor init — TinyMCE
+// raises a visible error notification for a missing langs/<code>.js.
+//
+// These translate the editor CHROME only. Content, the spellcheck dictionary
+// and Harper are untouched and stay English, which is why the control sits
+// under Editor rather than anywhere that reads as a global language setting.
+//
+// TinyMCE itself has no detection — "Auto" is ours, resolving the browser's
+// ordered language list against the shipped packs. The Auto row names what it
+// currently resolves to so the choice is not a guess. (Claude Code 2026-08-23)
+const optEditorLanguage = document.getElementById("opt-editor-language");
+if (optEditorLanguage) {
+    (async () => {
+        const scriptUrl = localStorage.getItem("mailx-tinymce-cdn") || "lib/tinymce/tinymce.min.js";
+        const { availableLanguages, resolveLanguage, languageLabel, LANGUAGE_AUTO, LANGUAGE_ENGLISH } = await import("./lib/tinymce-langs.js");
+        const available = await availableLanguages(scriptUrl);
+        const autoPick = resolveLanguage(LANGUAGE_AUTO, available);
+        const add = (value, text) => {
+            const o = document.createElement("option");
+            o.value = value;
+            o.textContent = text;
+            optEditorLanguage.appendChild(o);
+        };
+        add(LANGUAGE_AUTO, `Auto — ${languageLabel(autoPick)}`);
+        add(LANGUAGE_ENGLISH, languageLabel(LANGUAGE_ENGLISH));
+        for (const code of [...available].sort((a, b) => languageLabel(a).localeCompare(languageLabel(b)))) {
+            add(code, `${languageLabel(code)} (${code})`);
+        }
+        if (!available.length) {
+            optEditorLanguage.disabled = true;
+            optEditorLanguage.title = "No language packs found next to tinymce.min.js — English only";
+        }
+        let current = LANGUAGE_AUTO;
+        try {
+            current = localStorage.getItem("mailx-editor-language") || LANGUAGE_AUTO;
+        }
+        catch { /* */ }
+        try {
+            const settings = await getSettings();
+            if (typeof settings?.ui?.editorLanguage === "string" && settings.ui.editorLanguage)
+                current = settings.ui.editorLanguage;
+        }
+        catch { /* cache value stands */ }
+        optEditorLanguage.value = current;
+        // A pinned pack that is no longer shipped would leave the select on
+        // nothing at all; say so rather than showing a blank control.
+        if (!optEditorLanguage.value)
+            optEditorLanguage.value = LANGUAGE_AUTO;
+    })().catch((e) => console.error("[settings] editor-language menu failed:", e?.message || e));
+    optEditorLanguage.addEventListener("change", () => {
+        const value = optEditorLanguage.value;
+        // Synchronous cache write for the same reason as saveEditorSetting:
+        // compose reads it at module load, so the next compose-open is right.
+        try {
+            localStorage.setItem("mailx-editor-language", value);
+        }
+        catch { /* private mode */ }
+        getSettings().then((settings) => {
+            settings.ui = { ...settings.ui, editorLanguage: value };
+            saveSettings(settings);
+        }).catch((e) => console.error("[settings] editor language save failed:", e?.message || e));
+    });
+}
 // Save editor choice to server settings
 function saveEditorSetting(editor) {
     // Update the localStorage cache SYNCHRONOUSLY. compose.ts reads