@bobfrankston/rmfmail 1.2.319 → 1.2.320

package/.commitmsg

@@ -1,18 +1,14 @@
-Compose: typing after a pasted link no longer extends the link
+Compose (Quill): typing after a pasted or edited link is plain text
 
-Bob 2026-09-10: "there is still a problem in typing after pasting a link.
-It extends the link text rather than acting as regular text."
+Bob 2026-09-10: "you can risk the change for Quill since it is a known
+bug and I can test it" — the Quill counterpart of v1.2.319's TinyMCE fix.
 
-TinyMCE (his editor) turns a pasted URL into an anchor through its smart
-paste and leaves the caret INSIDE the anchor at the end of its text, so
-every character typed next became more link text while the target stayed
-the pasted URL — a link that no longer says what it does. Same when the
-clipboard carries an anchor copied from a page. New
-stepCaretOutOfPastedLink in client/compose/editor.ts: one tick after a
-paste, if the caret is collapsed at the very end of an anchor, it is set
-to just after the anchor; a caret mid-link or outside any link is left
-alone. Contained in mailx's TinyMCE wiring; rmf-tiny untouched.
-
-The Quill branch has the same shape (insertText with a link format, then
-setSelection at its end) and is not changed here — untested, and not the
-editor in use.
+Quill carries the format of the character before the caret into what is
+typed next, so every place that inserted a linked run and then set the
+caret at its end (the five paste paths — single-anchor clipboard, URL in
+an HTML clipboard, plain URL with and without a selection — and both
+branches of the Edit-link dialog) left the next keystrokes extending the
+link. One hoisted placeCaretAfterLink(index) in createQuillEditor now
+does setSelection + format("link", false) for all seven sites; format()
+on a collapsed selection sets the cursor format, which is how the
+toolbar's own toggle-then-type works. Untested on Quill — Bob tests.

package/.llm/trusted-lists.md

@@ -64,3 +64,4 @@ tests/trust.test.ts, .commitmsg, TODO.md C165, root version 1.2.311. Then rebuil
   click = edit in Google, right-click = Delete "<title>" (deleteCalendarEvent). Publishing as v1.2.317.
 - v1.2.318: New event Calendar picker (writable calendars via accessRole from listCalendars). Store-sync drain now passes calendarId to create/update/delete (was always "primary" — shared-calendar edits 404'd forever). Google: create/subscribe/share calendars on calendar.google.com; rmfmail lists the ones selected there.
 - v1.2.319: TinyMCE smart-paste left the caret INSIDE the pasted <a>; stepCaretOutOfPastedLink in client/compose/editor.ts moves it after the anchor one tick after paste. Quill path has the same shape (insertText link + setSelection) — untouched, Bob uses TinyMCE.
+- v1.2.320: Quill counterpart — placeCaretAfterLink(index) = setSelection + format("link", false) at 5 paste sites + 2 Edit-link dialog sites. Untested; Bob will switch editor to test.

package/client/compose/compose.bundle.js

@@ -4671,10 +4671,10 @@ function createQuillEditor(container2) {
     if (linkRange.length) {
       quill.deleteText(linkRange.index, linkRange.length);
       quill.insertText(linkRange.index, newText, { link: result.url });
-      quill.setSelection(linkRange.index + newText.length, 0);
+      placeCaretAfterLink(linkRange.index + newText.length);
     } else {
       quill.insertText(linkRange.index, newText, { link: result.url });
-      quill.setSelection(linkRange.index + newText.length, 0);
+      placeCaretAfterLink(linkRange.index + newText.length);
     }
   };
   const extraBindings = {
@@ -4978,7 +4978,7 @@ function createQuillEditor(container2) {
               q.deleteText(range.index, range.length);
             }
             q.insertText(range.index, text, { link: href });
-            q.setSelection(range.index + text.length, 0);
+            placeCaretAfterLink(range.index + text.length);
             return;
           }
         }
@@ -4991,10 +4991,10 @@ function createQuillEditor(container2) {
           const url = normalizeUrl(textOnly);
           if (range.length > 0) {
             q.formatText(range.index, range.length, "link", url);
-            q.setSelection(range.index + range.length, 0);
+            placeCaretAfterLink(range.index + range.length);
           } else {
             q.insertText(range.index, textOnly, { link: url });
-            q.setSelection(range.index + textOnly.length, 0);
+            placeCaretAfterLink(range.index + textOnly.length);
           }
           return;
         }
@@ -5010,10 +5010,10 @@ function createQuillEditor(container2) {
       const url = normalizeUrl(plain);
       if (range.length > 0) {
         q.formatText(range.index, range.length, "link", url);
-        q.setSelection(range.index + range.length, 0);
+        placeCaretAfterLink(range.index + range.length);
       } else {
         q.insertText(range.index, plain.trim(), { link: url });
-        q.setSelection(range.index + plain.trim().length, 0);
+        placeCaretAfterLink(range.index + plain.trim().length);
       }
       return;
     }
@@ -5033,6 +5033,10 @@ function createQuillEditor(container2) {
       q.clipboard.dangerouslyPasteHTML(range.index, markdownToEmailHtml(plain));
     }
   }, true);
+  function placeCaretAfterLink(index) {
+    q.setSelection(index, 0);
+    q.format("link", false);
+  }
   q.on("text-change", (delta, _old, source) => {
     if (source !== "user")
       return;