@bobfrankston/rmfmail 1.1.83 → 1.1.85

package/.commitmsg

@@ -1,5 +1,12 @@
-Viewer flag button: use a star glyph, matching the list star
+Flag star: single named colour token; drop unneeded VS-15 churn
 
-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.
+v1.1.84 already rendered both flag stars the same gold, so the
+variation-selector / font-variant-emoji changes were reverted — they
+fixed nothing and put invisible characters in source.
+
+Instead the gold/amber flag colour is now a documented token,
+--color-flag-star (oklch hue 60), in variables.css. Both the
+message-list row star (.ml-row.flagged .ml-flag) and the viewer toolbar
+Flag button (#btn-flag.flagged) reference it, so they can't drift, and
+the colour is named rather than a duplicated magic number. Star glyphs
+restored to plain literal ★/☆.

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}`);
   }
 });