@bobfrankston/rmfmail 1.1.87 → 1.1.88

package/.commitmsg

@@ -1,15 +1,11 @@
-Compose autocomplete at caret; instant discard; visible attachments
+Fix: highlighted row with an empty viewer (selection/preview desync)
 
-- Recipient autocomplete now matches the token the cursor is in, via
-  tokenSpanAtCaret(), instead of always the last token — typing a name
-  before an existing comma matched the wrong recipient. replaceCaretToken
-  replaces only that token and leaves following recipients intact.
+A list row could stay highlighted while the viewer showed "Select a
+message to read". releaseFocus() only un-selected the tracked focusedRow,
+so a stray .selected (multi-select remnant, or a row that kept the class
+through an incremental rerender) was left highlighted after the viewer
+had been cleared.
 
-- Compose Discard / X: the IMAP draft delete is now fire-and-forget so
-  the window closes instantly. It used to await the server round-trip,
-  leaving the window up for seconds ("eventually it did").
-
-- Attachment chip: clearly tinted background + accent border so it's not
-  easy to miss; clicking shows "Opening…" for >=600ms and ignores
-  re-clicks while in flight (the open hands off to the OS viewer with no
-  visible change in mailx, so it was being clicked repeatedly).
+releaseFocus() now strips every .selected row. focusRow() also sweeps
+stray .selected in single-select mode before highlighting its row, so
+the list can never show a highlight that doesn't match the viewer.

package/client/app.bundle.js

@@ -2522,6 +2522,13 @@ function rowKey(accountId, uid) {
   return `${accountId}:${uid}`;
 }
 function focusRow(row, opts = {}) {
+  const body = row.el.parentElement;
+  if (!body?.classList.contains("multi-select-on")) {
+    body?.querySelectorAll(".ml-row.selected").forEach((r) => {
+      if (r !== row.el)
+        r.classList.remove("selected");
+    });
+  }
   if (focusedRow && focusedRow !== row)
     focusedRow.setSelected(false);
   row.setSelected(true);
@@ -2559,9 +2566,9 @@ function scrollFocusedIntoView() {
     focusedRow.el.scrollIntoView({ block: "center" });
 }
 function releaseFocus() {
-  if (focusedRow)
-    focusedRow.setSelected(false);
   focusedRow = null;
+  const body = document.getElementById("ml-body");
+  body?.querySelectorAll(".ml-row.selected").forEach((r) => r.classList.remove("selected"));
   clearViewer();
   document.dispatchEvent(new CustomEvent("mailx-focus-changed", { detail: null }));
 }