@bobfrankston/rmfmail 1.1.98 → 1.1.99

package/.commitmsg

@@ -1,7 +1,16 @@
-Signature now applied to replies/forwards, not just new mail
+Fix duplicate drafts (unserialized autosave) + shift-click range select
 
-The new-format signature (acct.sig.text) was gated on "new mail only",
-so anyone using it got no signature on a reply or forward — only the
-legacy acct.signature field covered replies. Both formats now apply to
-new + reply + forward alike: above the quoted block on a reply/forward,
-at the end on new mail. Drafts still skip it (already in the saved body).
+Duplicate drafts: enqueueDraftPush fired imapManager.saveDraft immediately
+on every autosave (~every 2 s). An IMAP search-delete-append cycle takes
+longer than that, so pushes overlapped — each one's "find the stale copy"
+ran before a sibling's "append", neither saw the other's new copy, and
+the Drafts folder filled with dozens of duplicates. enqueueDraftPush now
+serializes per draftId: one push in flight at a time, and while it runs
+only the latest queued content is kept (intermediate autosaves are
+superseded — fewer IMAP ops too).
+
+Shift-click range select: selectRange now sets `multi-select-on` for a
+multi-row range. Without it, the focusRow() call right after (drives the
+viewer off the clicked row) ran its single-select sweep and wiped every
+row in the range except the clicked one — a regression from the v1.1.88
+highlight-desync fix.

package/client/app.bundle.js

@@ -2633,6 +2633,8 @@ function selectRange(from, to) {
     return;
   const lo = Math.min(fromIdx, toIdx);
   const hi = Math.max(fromIdx, toIdx);
+  if (hi > lo)
+    body.classList.add("multi-select-on");
   for (let i = lo; i <= hi; i++)
     rows[i].classList.add("selected");
 }