@bobfrankston/rmfmail 1.2.318 → 1.2.320

package/.commitmsg

@@ -1,19 +1,14 @@
-New event: choose the calendar; pushes go to the event's own calendar
+Compose (Quill): typing after a pasted or edited link is plain text
 
-Bob 2026-09-10: "when creating an event I should be able to specify
-which calendar. I want to start making more use of shared calendars."
+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.
 
-The New event dialog gains a Calendar picker listing the calendars
-Google says the reader can write to (accessRole owner or writer, now
-carried by listCalendars → getCalendars), personal first, shown only
-when there is more than one; the last choice is remembered per machine.
-The status line names the calendar the event went to.
-
-Underneath, a real bug: the store-sync drain dropped calendarId and sent
-every create, update and delete to "primary". A create could not land
-anywhere else, and an update or delete of an event on a shared calendar
-was a 404 retried on every poll. createCalendarEventLocal takes
-calendarId, the row and the queue payload carry it, update/delete
-payloads carry the row's, and the drain passes it to all three Google
-calls. Rows pulled before today already hold the id the pull recorded.
-mailx-types 0.1.95, mailx-service 0.1.37.
+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

@@ -63,3 +63,5 @@ tests/trust.test.ts, .commitmsg, TODO.md C165, root version 1.2.311. Then rebuil
   Refresh button + window focus call refreshCalendarNow. Tooltips on all sidebar controls; event rows:
   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;
@@ -5320,11 +5324,34 @@ async function createTinyMceEditor2(container2, opts = {}) {
     const body = native?.getBody?.();
     if (native && body) {
       wireMarkdownPaste(body, (html) => native.insertContent(html));
+      stepCaretOutOfPastedLink(native);
     }
   } catch {
   }
   return ed;
 }
+function stepCaretOutOfPastedLink(native) {
+  native.on("paste", () => {
+    setTimeout(() => {
+      try {
+        const rng = native.selection.getRng();
+        if (!rng || !rng.collapsed)
+          return;
+        const node = native.selection.getNode();
+        const a = node?.closest?.("a[href]");
+        if (!a)
+          return;
+        const rest = rng.cloneRange();
+        rest.setEnd(a, a.childNodes.length);
+        if (rest.toString().trim())
+          return;
+        native.selection.setCursorLocation(a.parentNode, native.dom.nodeIndex(a) + 1);
+      } catch (e) {
+        console.warn("[editor] could not move the caret past the pasted link:", e?.message || e);
+      }
+    }, 0);
+  });
+}
 
 // client/compose/compose.ts
 init_api_client();