@bobfrankston/rmfmail 1.2.195 → 1.2.196

package/client/app.ts

@@ -94,13 +94,17 @@ propagateAppName();
         if (e.key === "F5" || e.key === "F3") return true;
         if ((e.ctrlKey || e.metaKey) && (e.key === "r" || e.key === "R")) return true;
         // Browser accelerators bound to Ctrl-letter combos that mailx
-        // doesn't want to forward: P (print), S (save page), U (view
-        // source), J (downloads), L (focus address bar — n/a in WebView
-        // but harmless), G (find next). Note: O is "open file" in
-        // browsers — mailx uses Ctrl+O nowhere meaningful, so block.
+        // doesn't want to forward: S (save page), U (view source),
+        // J (downloads), L (focus address bar — n/a in WebView but
+        // harmless), G (find next). Note: O is "open file" in browsers —
+        // mailx uses Ctrl+O nowhere meaningful, so block. P is NOT here:
+        // mailx's own Ctrl+P handler prints the focused letter and
+        // preventDefaults the browser dialog itself — listing it here
+        // stopPropagation'd the event before that handler could run, which
+        // is why the print shortcut was dead (Bob 2026-07-30).
         if (e.ctrlKey || e.metaKey) {
             const k = e.key.toLowerCase();
-            if (["p", "s", "u", "j", "l", "g", "o"].includes(k)) return true;
+            if (["s", "u", "j", "l", "g", "o"].includes(k)) return true;
         }
         // Alt+Left / Alt+Right = browser back / forward. No use in mailx.
         if (e.altKey && (e.key === "ArrowLeft" || e.key === "ArrowRight")) return true;
@@ -3746,13 +3750,17 @@ document.addEventListener("keydown", (e) => {
         openCompose("new");
     }
     // Ctrl+P = Print the focused message. preventDefault stops the host
-    // browser from printing the whole app window instead of the letter.
+    // browser from printing the whole app window instead of the letter —
+    // UNCONDITIONALLY, even with focus in a text field: the WebView's
+    // whole-app print dialog is never what the user wants, and ^P should
+    // print the current letter regardless of where the caret sits.
     if (e.ctrlKey && (e.key === "p" || e.key === "P") && !e.shiftKey && !e.altKey) {
-        const t = e.target as HTMLElement | null;
-        const inText = t && (t.tagName === "INPUT" || t.tagName === "TEXTAREA" || t.isContentEditable);
-        if (!inText) {
-            e.preventDefault();
+        e.preventDefault();
+        if (getCurrentMessage()) {
             printCurrentMessage();
+        } else {
+            const s = document.getElementById("status-sync");
+            if (s) s.textContent = "Nothing to print — select a message first.";
         }
     }
     // Ctrl+T = new view tab. The strip is hidden with a single tab (no wasted

package/client/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-client",
-  "version": "1.0.24",
+  "version": "1.0.25",
   "private": true,
   "type": "module",
   "scripts": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/rmfmail",
-  "version": "1.2.195",
+  "version": "1.2.196",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",