@bobfrankston/rmfmail 1.1.97 → 1.1.99

package/.commitmsg

@@ -1,15 +1,16 @@
-Reply scrolls to top; Ctrl+A no longer hijacked in text fields
+Fix duplicate drafts (unserialized autosave) + shift-click range select
 
-- Reply/forward: setCursor(0) now scrolls the editor viewport to the top
-  (editor.getWin().scrollTo(0,0)), so the caret above the quoted block is
-  the line you're looking at instead of being left scrolled down inside
-  the quote.
+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).
 
-- Ctrl+A: the message-list select-all guard used `.closest(".message-list,
-  .ml-body, body)` — `body` always matches, so Ctrl+A in the search box
-  (or any input) selected every message instead of the box's text. Now it
-  returns early when focus is in an INPUT/TEXTAREA/SELECT/contenteditable,
-  so Ctrl+A selects (and lets you clear) the search field normally.
-
-- docs/rmf-tiny.md: marked implemented; added a running "Known TinyMCE
-  quirks" log (caret/scroll, link-extend, spellcheck, <p>-to-<br>).
+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");
 }