@bobfrankston/rmfmail 1.2.195 → 1.2.196

package/client/app.js

@@ -93,13 +93,17 @@ window.__btick && window.__btick("app.ts module body executing");
         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))
+            if (["s", "u", "j", "l", "g", "o"].includes(k))
                 return true;
         }
         // Alt+Left / Alt+Right = browser back / forward. No use in mailx.
@@ -3931,14 +3935,20 @@ 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;
-        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
     // band), so this is how the second tab gets opened; once 2+ exist the