@bobfrankston/rmfmail 1.2.319 → 1.2.321

package/.commitmsg

@@ -1,18 +1,10 @@
-Compose: typing after a pasted link no longer extends the link
+Compose: a thin 3 cm rule above the signature instead of "-- "
 
-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."
-
-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.
+Bob 2026-09-10: "you seem to be putting a -- above the signature. If I
+want it it should be part of my HTML. If you do it it should be a line
+rather than -- and maybe three cm or an inch." The signature block in
+compose.ts hard-coded the "-- " delimiter above acct.sig / acct.signature.
+It is now a left-aligned <hr> 3 cm wide, 1 px, grey, inline-styled since
+no stylesheet travels with mail. Anything else belongs in the signature
+HTML itself. Trade-off noted to Bob: other clients use "-- " to trim a
+quoted signature; without it some will quote it back.

package/.llm/trusted-lists.md

@@ -64,3 +64,5 @@ 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.
+- v1.2.321: signature block uses a 3cm <hr> instead of '-- ' (compose.ts sigBlock).

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;
@@ -6244,7 +6248,8 @@ function applyInit(init) {
       sigHtml = acct.signature;
     }
     if (sigHtml) {
-      const sigBlock = `<br><br>-- <br>${sigHtml}`;
+      const sigRule = '<hr style="width:3cm;margin:0.8em 0 0.5em 0;border:0;border-top:1px solid #999;">';
+      const sigBlock = `<br>${sigRule}${sigHtml}`;
       bodyToRender = isReplyForward ? `<br>${sigBlock}<br>${bodyToRender}` : `${bodyToRender}${sigBlock}`;
     }
   }