@bobfrankston/rmfmail 1.1.89 → 1.1.91

package/.commitmsg

@@ -1,8 +1,11 @@
-Spellcheck: disable the native checker so only nspell runs
+Fix: typing after a URL no longer swallows text into the link
 
-The compose editor had BOTH the native Chromium/WebView2 spellchecker and
-mailx's own nspell checker active — double red underlines everywhere
-("why so much red twiddle") and right-click popping the native
-suggestion menu instead of ours. wireSpellcheck now forces
-spellcheck="false" on the editor body (with a MutationObserver to keep
-it off), so nspell is the single source of squiggles and suggestions.
+In the TinyMCE editor, typing or pasting a URL auto-linked it, but the
+caret stayed inside the <a>, so every following character was appended
+into the link text — a whole sentence turned into one link.
+
+rmf-tiny's adapter now has a keypress handler: when a printable key is
+pressed with the caret collapsed at the trailing edge of an <a>, the
+caret hops to just after the <a> first, so the new text lands outside
+the link. Covers autolink, pasted URLs, and hand-made links.
+(client/lib/rmf-tiny.js rebuilt from the adapter source.)

package/client/compose/compose.bundle.js

@@ -742,6 +742,30 @@ async function createTinyMceEditor(container2, opts = {}) {
             applyZoom();
           }
         });
+        ed.on("keypress", (e) => {
+          if (e.ctrlKey || e.altKey || e.metaKey)
+            return;
+          const sel = ed.selection;
+          const rng = sel.getRng();
+          if (!rng || !rng.collapsed)
+            return;
+          const a = ed.dom.getParent(sel.getNode(), "a");
+          if (!a)
+            return;
+          let tail;
+          try {
+            tail = rng.cloneRange();
+            tail.setEndAfter(a);
+          } catch {
+            return;
+          }
+          if (tail.toString().length > 0)
+            return;
+          const after = ed.getDoc().createRange();
+          after.setStartAfter(a);
+          after.collapse(true);
+          sel.setRng(after);
+        });
         ed.on("init", () => {
           try {
             const body = ed.getBody();