@bobfrankston/rmfmail 1.1.83 → 1.1.84

package/.commitmsg

@@ -1,5 +1,6 @@
-Viewer flag button: use a star glyph, matching the list star
+Viewer Flag button reflects flagged state, like the list star
 
-The toolbar Flag button showed a ⚑ while the message-list row uses
-★/☆ for the same flag state. Changed the toolbar glyph to ★ so the
-two are visibly the same action.
+The toolbar Flag button is now ☆ in the default colour when the viewed
+message is unflagged and a gold ★ when flagged — same glyph + colour
+behaviour as the message-list row star. updateFlagButton() runs on
+mailx-message-shown and after every toggle (including the error revert).

package/client/app.bundle.js

@@ -7447,17 +7447,28 @@ document.addEventListener("mailx-moved", (e) => {
 document.getElementById("btn-delete")?.addEventListener("click", deleteSelection);
 document.getElementById("btn-tb-delete")?.addEventListener("click", deleteSelection);
 document.getElementById("btn-tb-spam")?.addEventListener("click", spamSelectedMessages);
+function updateFlagButton() {
+  const btn = document.getElementById("btn-flag");
+  if (!btn) return;
+  const sel = getCurrentFocused();
+  const yes = !!sel && flaggedOf(sel);
+  btn.classList.toggle("flagged", yes);
+  btn.textContent = yes ? "\u2605" : "\u2606";
+}
+document.addEventListener("mailx-message-shown", updateFlagButton);
 document.getElementById("btn-flag")?.addEventListener("click", async () => {
   const sel = getCurrentFocused();
   if (!sel) return;
   const wasFlagged = flaggedOf(sel);
   setFlagged(sel, !wasFlagged);
+  updateFlagButton();
   try {
     await updateFlags(sel.accountId, sel.uid, sel.flags);
     updateMessageFlags(sel.accountId, sel.uid, sel.flags);
     setRowFlagged(sel.accountId, sel.uid, !wasFlagged);
   } catch (e) {
     setFlagged(sel, wasFlagged);
+    updateFlagButton();
     console.error(`Flag toggle failed: ${e.message}`);
   }
 });