@bobfrankston/rmfmail 1.0.681 → 1.0.686

package/client/app.js

@@ -1133,8 +1133,19 @@ function sanitizeQuotedBody(msg) {
         // like `<!--` / `-->` / `<!` in plain-text bodies can be misread
         // by the parser. Per Bob 2026-05-12: "not just ugly, it breaks
         // the HTML." Trivial source-clutter is the lesser evil.
+        //
+        // CRLF → <br>. `white-space:pre-wrap` alone is not enough: TinyMCE
+        // (and other HTML editors) normalize text-node whitespace when
+        // setContent ingests the HTML, collapsing `\n` to spaces BEFORE
+        // CSS runs. The literal `<br>` survives the normalization, so we
+        // get one line break per source line whether the editor preserves
+        // raw whitespace or not. pre-wrap stays as belt-and-braces and to
+        // keep multi-space runs (alignment, indented quote-markers like
+        // `> > >`) visible.
         const escaped = String(msg.bodyText || "")
-            .replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;");
+            .replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;")
+            .replace(/\r\n?/g, "\n")
+            .replace(/\n/g, "<br>");
         return `<div style="white-space:pre-wrap;font-family:inherit;margin:0">${escaped}</div>`;
     }
     let body = msg.bodyHtml;