@bongos/core 1.21.163 → 1.21.165

package/modules/copy-desk/page-status.js

@@ -691,6 +691,8 @@ function approvalFor({ page, pageId, rounds, round, builderId, isArtist }) {
   const applied = pages.parseAppliedBlock(round.description);
   const refusedBy = new Map();
   if (applied.ok) for (const r of applied.applied.refused || []) if (r && r.key) refusedBy.set(String(r.key), String(r.code || 'refused'));
+  const pinnedBy = new Map();
+  if (applied.ok) for (const p of applied.applied.pinned_tests || []) if (p && p.key && Array.isArray(p.tests)) pinnedBy.set(String(p.key), p.tests.map(String));
   const lines = batch.ok ? batch.batch.lines.map((l) => ({
     key: l.key,
     section: l.section == null ? null : l.section,
@@ -698,6 +700,7 @@ function approvalFor({ page, pageId, rounds, round, builderId, isArtist }) {
     after: l.after,
     placement: l.placement || null,
     refused: refusedBy.get(String(l.key)) || null,
+    ...(pinnedBy.has(String(l.key)) ? { pinned_by: pinnedBy.get(String(l.key)) } : {}),
   })) : [];
   const shipped = (rounds || []).filter((r) => r.status === 'shipped');
   const now = countsFor(rounds);

package/modules/copy-desk/pages.js

@@ -164,6 +164,11 @@ function composeAppliedBlock(a) {
     reading_hash_after: a.reading_hash_after,
     lines_after: (a.lines_after || []).map(String),
     refused: (a.refused || []).map((x) => ({ key: x.key, code: x.code })),
+    // task 1004775: tests that still assert a rewritten line's old words, named
+    // at apply time so the artist sees them before approving, not CI after.
+    ...(Array.isArray(a.pinned_tests) && a.pinned_tests.length
+      ? { pinned_tests: a.pinned_tests.map((p) => ({ key: p.key, tests: (p.tests || []).map(String) })) }
+      : {}),
   });
 }
 

package/modules/hall-ui/public/approval-queue-lib.js

@@ -119,6 +119,10 @@
       marked: after && !line.refused,
       aside: line.refused ? refusedWords(line.refused) : (after ? `was: ${line.before}` : `becomes: ${line.after}`),
       refused: !!line.refused,
+      // task 1004775: a test still asserts the old words, so it fails once this lands.
+      ...(!line.refused && Array.isArray(line.pinned_by) && line.pinned_by.length
+        ? { pinned: `A test still checks the old words: ${line.pinned_by.join(', ')}. It needs updating when this ships.` }
+        : {}),
     };
   }
 

package/modules/hall-ui/public/approval-queue.css

@@ -226,6 +226,8 @@ html.aq-open { overflow: hidden; }
 }
 .aq-line__aside { font-size: 13px; line-height: 1.45; color: var(--aq-ink-faint); overflow-wrap: anywhere; }
 .aq-line--refused .aq-line__aside { color: var(--aq-link); }
+/* task 1004775: a test that still checks the old words. */
+.aq-line__pinned { font-size: 13px; line-height: 1.45; color: var(--aq-link); overflow-wrap: anywhere; }
 .aq-lines__none { margin: 0; font-size: 14px; color: var(--aq-ink-faint); }
 
 /* The decision row. */

package/modules/hall-ui/public/approval-queue.js

@@ -152,6 +152,7 @@
           ${r.section ? `<span class="aq-line__section">${esc(r.section)}</span>` : ''}
           <span class="aq-line__text">${r.marked ? `<mark class="aq-mark">${words(r.text)}</mark>` : words(r.text)}</span>
           <span class="aq-line__aside">${words(r.aside)}</span>
+          ${r.pinned ? `<span class="aq-line__pinned">${esc(r.pinned)}</span>` : ''}
         </li>`).join('')}</ol>`;
     }
 

package/modules/hall-ui/public/builders.js

@@ -929,7 +929,7 @@
 
   // ---- Home "Next up" slice (task 2184) -------------------------------------
 
-  // Fill the top of the "Your work" card with the first two rows of the
+  // Fill the top of the "Your work" card with the first five rows of the
   // claimable queue (server order: priority, then calibrated estimate) so Home
   // answers "what's my next action". Best-effort: a primer-gated 403, an auth
   // hiccup, or an empty queue hides the slice — Home never shows an error for
@@ -947,7 +947,8 @@
       // already orders by priority then calibrated estimate, so the first two rows ARE
       // the top two. `claimable_total` comes back so the foot can say how many are
       // waiting instead of leaving two rows to imply there are only two.
-      const resp = await getJSON(`${API}/tasks/claimable?limit=2`, { softAuth: true });
+      // task 1004774: five rows, level with the five Recently shipped rows beside it.
+      const resp = await getJSON(`${API}/tasks/claimable?limit=5`, { softAuth: true });
       // task 1003464: the greeting's "N tasks are claimable right now" — rank-scoped,
       // so it is THIS builder's number. It reads claimable_total rather than the length
       // of a fully-downloaded feed: the same number, without the wire cost that this
@@ -959,7 +960,8 @@
         ? claimableTotal
         : (resp && Array.isArray(resp.claimable) ? resp.claimable.length : null);
       renderGreeting();
-      const rows = resp && Array.isArray(resp.claimable) ? resp.claimable : [];
+      // capped as well as asked for: a server that ignores ?limit must not grow the card.
+      const rows = resp && Array.isArray(resp.claimable) ? resp.claimable.slice(0, 5) : [];
       if (!rows.length) { wrap.hidden = true; if (subhead) subhead.hidden = true; return; }
       const total = Number(resp.claimable_total);
       const cur = (window.__BRANDING__ && window.__BRANDING__.currency && window.__BRANDING__.currency.label) || 'credits';
@@ -1175,7 +1177,7 @@
       getJSON(`${API}/me/recent-ships?limit=all`),
       getJSON(`${API}/versions`),
       getJSON(`${API}/public/leaderboard`), // task 1004437: the list with each builder's roles
-      getJSON(`${API}/public/progress`), // task 1003464: "<version> is N percent shipped"
+      getJSON(`${API}/public/progress`), // task 1003464: "<version> is N percent complete"
     ]);
 
     if (progress.status === 'fulfilled') {

package/modules/hall-ui/public/craft-mode.js

@@ -34,7 +34,6 @@
     governing: '<path d="M2 6 8 2.5 14 6M3.5 6.5v6M6.5 6.5v6M9.5 6.5v6M12.5 6.5v6M2 13.5h12"/>',
     brush: '<path d="M13.6 2.4 7.4 8.6"/><path d="M7.4 8.6c-1.3-.7-3-.1-3.4 1.5-.3 1.3-.8 2.2-1.9 2.9 2.4.7 4.6.1 5.6-1.4.6-.9.5-2.1-.3-3Z"/>',
     note: '<path d="M9.5 2.5h-6v11h9v-6"/><path d="M5.5 8h5M5.5 10.5h3.5"/><path d="M12.5 1.5v4M10.5 3.5h4"/>',
-    write: '<path d="M2.5 13.5h11"/><path d="M10.4 2.6l2.9 2.9-6.4 6.4H4V9z"/>',
     modes: '<rect x="2.5" y="2.5" width="4.5" height="4.5" rx="1"/><rect x="9" y="2.5" width="4.5" height="4.5" rx="1"/><rect x="2.5" y="9" width="4.5" height="4.5" rx="1"/><rect x="9" y="9" width="4.5" height="4.5" rx="1"/>',
   };
   // The four crafts. `short` is the switch's label; `line` is the
@@ -342,8 +341,9 @@
     artistry: [
       // Named "Artist review" since task 1004499: the public site's button
       // (public-landing assets/artist-review.js) opens this same panel.
-      // `menu`: hovering it opens the review-mode menu (task 1004686, below).
-      { id: 'palette', icon: 'artistry', short: 'Review', label: 'Artist review: write feedback on this page', script: 'page-notes.js', api: 'OTBPageNotes', menu: true },
+      // `docked` (task 1004771): no top-bar button. The panel itself sits on
+      // the right edge behind a small tab, and carries the review-mode choice.
+      { id: 'palette', icon: 'artistry', short: 'Review', label: 'Artist review: write feedback on this page', script: 'page-notes.js', api: 'OTBPageNotes', docked: true },
       // Task 1004497: this page's words, edited where they stand as a proposed wording change.
       { id: 'paintbrush', icon: 'brush', short: 'Edit words', label: 'Edit this page\'s words', script: 'paintbrush.js', api: 'OTBPaintbrush' },
     ],
@@ -393,21 +393,24 @@
     b.setAttribute('aria-label', t.label);
     b.title = t.label;
     b.innerHTML = icon(t.icon, 'craft-tool__icon') + '<span class="craft-tool__label">' + esc(t.short) + '</span>';
-    b.addEventListener('click', function (e) {
-      // A phone has no hover, so there a tap on Review opens its menu instead.
-      if (t.menu && narrow()) { toggleReviewMenu(e && e.detail === 0); return; }
-      openTool(t, b);
+    b.addEventListener('click', function () { openTool(t, b); });
+    return b;
+  }
+  function toolNode(t) { return document.getElementById('craft-tool-' + t.id); }
+  // A docked tool has no button: in its mode the panel is docked at the edge,
+  // out of it the panel goes. Only a tool already loaded needs closing.
+  function applyDocked(mode) {
+    var want = toolsFor(mode);
+    for (var m in TOOLS) TOOLS[m].forEach(function (t) {
+      if (!t.docked) return;
+      if (want.indexOf(t) !== -1) loadTool(t).then(function (api) { if (api.dock) api.dock(); }, function () { /* the page still works */ });
+      else if (window[t.api] && window[t.api].close) window[t.api].close();
     });
-    return t.menu ? reviewGroup(t, b) : b;
-  }
-  // The node a tool occupies in the bar: its button, or Review's whole group.
-  function toolNode(t) {
-    var b = document.getElementById('craft-tool-' + t.id);
-    return b && t.menu && b.parentNode && b.parentNode.id === REVIEW_GROUP ? b.parentNode : b;
   }
   function applyTools(mode) {
     var sw = document.getElementById('craft-switch');
-    var own = toolsFor(mode);
+    applyDocked(mode);
+    var own = toolsFor(mode).filter(function (t) { return !t.docked; });
     var old = document.querySelectorAll ? document.querySelectorAll('[data-craft-tool]') : [];
     for (var i = old.length - 1; i >= 0; i--) {
       if (own.some(function (t) { return t.id === old[i].getAttribute('data-craft-tool'); })) continue;
@@ -429,26 +432,19 @@
       var el = document.getElementById('craft-tool-' + t.id);
       if (el && el.classList) el.classList.toggle('craft-tool--guest', !!review && TOOLS.artistry.indexOf(t) === -1);
     });
-    var on = document.getElementById('craft-tool-palette');
-    if (on) {
-      on.classList.toggle('craft-tool--reviewing', !!review);
-      var word = on.querySelector && on.querySelector('.craft-tool__label');
-      if (word) word.textContent = review ? 'Reviewing' : 'Review';
-    }
   }
 
   // ---------- artist review mode (task 1004686) ----------
-  // Hovering Artistry's Review button opens a menu of the four experiences
-  // (owner, 2026-10-09, in the owner's order and words). Picking one makes the hall
+  // The review panel's Review mode row (page-notes.js, task 1004771) offers the four
+  // experiences (owner, 2026-10-09, in the owner's order and words). Picking one makes the hall
   // wear that mode on every page — its tabs, its look, its tools — the way a
   // builder in it sees it, while the artist keeps Artistry's tools and small
   // sky-style doodles are scribbled into the top bar's empty space (craft-review.js draws them,
-  // loaded only in review mode; craft-review.css styles it and the menu).
+  // loaded only in review mode; craft-review.css styles it).
   // The artist's own mode stays Artistry: the experience is this session's
   // alone (sessionStorage), and choosing any mode from the switch or the
   // picker ends it. Cosmetic like the mode: a tab the gate hides stays hidden.
   var REVIEW_KEY = 'hall-craft-review';
-  var REVIEW_GROUP = 'craft-review';
   var EXPERIENCES = [
     { id: 'ideating', label: 'Ideator experience' },
     { id: 'governing', label: 'Governing experience' },
@@ -478,138 +474,6 @@
     apply('artistry');
   }
 
-  // The menu. Hover opens it where a pointer can hover; the chevron (or, on a
-  // phone, Review itself) opens it by click, tap or keyboard. Built on every
-  // open, so the checked experience is always the live one.
-  var closeTimer = null;
-  // Hover is a mouse's alone: a touch also fires pointer events, and a laptop
-  // with a touch screen can report no hover at all, so ask the pointer itself.
-  function byMouse(e) { return !!e && e.pointerType === 'mouse'; }
-  function reviewMenuHtml() {
-    var items = '<button class="craft-review__item" type="button" role="menuitem" data-review-act="write">' +
-      icon('write', 'craft-review__icon') + '<span>Write feedback on this page</span></button>' +
-      '<p class="craft-review__head" role="presentation">Review mode</p>';
-    items += EXPERIENCES.map(function (x) {
-      var on = review === x.id;
-      return '<button class="craft-review__item" type="button" role="menuitemradio" aria-checked="' + (on ? 'true' : 'false') + '" data-review-pick="' + x.id + '">' +
-        icon(x.id, 'craft-review__icon') + '<span>' + esc(x.label) + '</span>' +
-        (on ? '<svg class="craft-review__check" viewBox="0 0 16 16" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><path d="M3.5 8.5l3 3 6-7"/></svg>' : '') +
-        '</button>';
-    }).join('');
-    if (review) {
-      items += '<button class="craft-review__item craft-review__item--stop" type="button" role="menuitem" data-review-act="stop">' +
-        '<svg class="craft-review__icon" viewBox="0 0 16 16" fill="none" stroke="currentColor" stroke-width="1.4" stroke-linecap="round" aria-hidden="true"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7"/></svg><span>Stop reviewing</span></button>';
-    }
-    return items;
-  }
-  function menuEl() { return document.getElementById(REVIEW_GROUP + '-menu'); }
-  function menuItems() {
-    var m = menuEl();
-    return m && m.querySelectorAll ? Array.prototype.slice.call(m.querySelectorAll('[role^="menuitem"]')) : [];
-  }
-  function openReviewMenu(focusFirst) {
-    var m = menuEl(), more = document.getElementById(REVIEW_GROUP + '-more');
-    if (!m) return;
-    clearTimeout(closeTimer);
-    if (m.hasAttribute('hidden')) {
-      m.innerHTML = reviewMenuHtml();
-      m.removeAttribute('hidden');
-      if (more) more.setAttribute('aria-expanded', 'true');
-      document.addEventListener('click', onOutside, true);
-    }
-    if (focusFirst) {
-      var first = menuItems()[0];
-      if (first && first.focus) first.focus();
-    }
-  }
-  function closeReviewMenu(refocus) {
-    var m = menuEl(), more = document.getElementById(REVIEW_GROUP + '-more');
-    clearTimeout(closeTimer);
-    document.removeEventListener('click', onOutside, true);
-    if (!m || m.hasAttribute('hidden')) return;
-    m.setAttribute('hidden', '');
-    if (more) more.setAttribute('aria-expanded', 'false');
-    if (refocus) {
-      var back = narrow() ? document.getElementById('craft-tool-palette') : more;
-      if (back && back.focus) back.focus();
-    }
-  }
-  function toggleReviewMenu(focusFirst) {
-    var m = menuEl();
-    if (m && !m.hasAttribute('hidden')) closeReviewMenu(false); else openReviewMenu(focusFirst);
-  }
-  function onOutside(e) {
-    var g = document.getElementById(REVIEW_GROUP);
-    if (g && g.contains && !g.contains(e.target)) closeReviewMenu(false);
-  }
-  function onMenuClick(e) {
-    var btn = e.target.closest && e.target.closest('button');
-    if (!btn) return;
-    var pick = btn.getAttribute('data-review-pick');
-    var act = btn.getAttribute('data-review-act');
-    if (!pick && !act) return;
-    closeReviewMenu(false);
-    if (pick) startReview(pick);
-    else if (act === 'stop') stopReview();
-    else if (act === 'write') openTool(toolFor('palette'), document.getElementById('craft-tool-palette'));
-  }
-  // The menu keys: arrows move, Home and End jump, Escape closes back to the
-  // button that opened it, Tab closes and lets focus move on.
-  function onMenuKey(e) {
-    var items = menuItems();
-    var at = items.indexOf(document.activeElement);
-    var to = null;
-    if (e.key === 'ArrowDown') to = (at + 1) % items.length;
-    else if (e.key === 'ArrowUp') to = (at - 1 + items.length) % items.length;
-    else if (e.key === 'Home') to = 0;
-    else if (e.key === 'End') to = items.length - 1;
-    else if (e.key === 'Escape') { e.preventDefault(); closeReviewMenu(true); return; }
-    else if (e.key === 'Tab') { closeReviewMenu(false); return; }
-    if (to === null || !items[to]) return;
-    e.preventDefault();
-    items[to].focus();
-  }
-  function reviewGroup(t, b) {
-    var g = document.createElement('span');
-    g.id = REVIEW_GROUP;
-    g.className = 'craft-review';
-    var more = document.createElement('button');
-    more.type = 'button';
-    more.id = REVIEW_GROUP + '-more';
-    more.className = 'craft-review__more';
-    more.setAttribute('aria-haspopup', 'menu');
-    more.setAttribute('aria-expanded', 'false');
-    more.setAttribute('aria-controls', REVIEW_GROUP + '-menu');
-    more.setAttribute('aria-label', 'Review mode: see the hall as another mode');
-    more.title = 'Review mode: see the hall as another mode';
-    more.innerHTML = '<svg viewBox="0 0 12 12" fill="none" stroke="currentColor" stroke-width="1.5" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><path d="M3 4.5 6 7.5 9 4.5"/></svg>';
-    more.addEventListener('click', function (e) { toggleReviewMenu(e && e.detail === 0); });
-    more.addEventListener('keydown', function (e) {
-      if (e.key === 'ArrowDown') { e.preventDefault(); openReviewMenu(true); }
-    });
-    var menu = document.createElement('div');
-    menu.id = REVIEW_GROUP + '-menu';
-    menu.className = 'craft-review__menu';
-    menu.setAttribute('role', 'menu');
-    menu.setAttribute('aria-label', 'Review mode');
-    menu.setAttribute('hidden', '');
-    menu.addEventListener('click', onMenuClick);
-    menu.addEventListener('keydown', onMenuKey);
-    b.addEventListener('keydown', function (e) {
-      if (e.key === 'ArrowDown') { e.preventDefault(); openReviewMenu(true); }
-    });
-    g.addEventListener('pointerenter', function (e) { if (byMouse(e)) openReviewMenu(false); });
-    g.addEventListener('pointerleave', function (e) {
-      if (!byMouse(e)) return;
-      clearTimeout(closeTimer);
-      closeTimer = setTimeout(function () { closeReviewMenu(false); }, 220);
-    });
-    g.appendChild(b);
-    g.appendChild(more);
-    g.appendChild(menu);
-    return g;
-  }
-
   // The scribble wall: the top bar's empty space, doodled on while reviewing.
   // craft-review.js draws it (decorative, aria-hidden); every motion is CSS,
   // so the reduced-motion kill switch leaves three finished doodles standing.
@@ -637,7 +501,7 @@
     loadTool(wallTool).then(function (api) {
       var live = document.getElementById(WALL);
       if (live === wall && review === mode) wall.innerHTML = api.wallHtml(mode);
-    }, function () { /* a bare wall; the menu still says you are reviewing */ });
+    }, function () { /* a bare wall; the review panel's mode row still says which */ });
   }
   // sessionStorage: this session's mode, or that the picker was already shown.
   var MODE_KEY = 'hall-craft';
@@ -703,7 +567,7 @@
   function choose(mode, keep) {
     mode = valid(mode);
     if (!mode || !offered(mode)) return;
-    if (review) { drop('sessionStorage', REVIEW_KEY); review = null; closeReviewMenu(false); }
+    if (review) { drop('sessionStorage', REVIEW_KEY); review = null; }
     set('sessionStorage', MODE_KEY, mode);
     set('sessionStorage', ASKED_KEY, '1');
     if (keep === true) set('localStorage', KEEP_KEY, mode);
@@ -909,6 +773,16 @@
     review: startReview,
     stopReview: stopReview,
     EXPERIENCES: EXPERIENCES.map(function (x) { return x.id; }),
+    // The review panel's Review mode row (page-notes.js) draws the four
+    // experiences; `short` is the mode switch's own word for each, so the
+    // row and the switch never name one mode two ways.
+    experiences: function () {
+      return EXPERIENCES.map(function (x) {
+        var m = MODES.filter(function (y) { return y.id === x.id; })[0];
+        return { id: x.id, label: x.label, short: m ? m.short : x.label };
+      });
+    },
+    iconHtml: icon,
     open: openPicker,
     ALWAYS_TABS: ALWAYS_TABS.slice(),
     tabsFor: tabsFor,

package/modules/hall-ui/public/craft-review.css

@@ -1,10 +1,10 @@
-/* craft-review.css — artist review mode (task 1004686; craft-mode.js draws the
-   menu, craft-review.js the wall). Imported by style.css, beside page-notes.css.
+/* craft-review.css — artist review mode (task 1004686; craft-review.js draws
+   the wall). Imported by style.css, beside page-notes.css.
 
-   The Review button becomes a split pill: Review still opens the feedback doc,
-   and the chevron (or, where a pointer can hover, the whole pill) opens a menu
-   of the four experiences. While an artist reviews one, small black-and-white
-   doodles in the sky's hand are scribbled into the top bar's empty space.
+   The experience under review is picked in the review panel's own Review mode
+   row (page-notes.js, task 1004771); the top bar has no Review button any more.
+   While an artist reviews one, small black-and-white doodles in the sky's hand
+   are scribbled into the top bar's empty space.
 
    Every colour is a token (tests/hall_audit.mjs holds that). Every motion is
    CSS on transform and opacity, and the hall's reduced-motion kill switch
@@ -13,126 +13,13 @@
    An @import lands BEFORE style.css's own rules, so a rule here that restyles
    .craft-tool must out-rank it (two classes), or style.css silently wins. */
 
-/* ---------- the split pill ---------- */
-.craft-review {
-  position: relative;
-  display: inline-flex;
-  align-items: center;
-  flex-shrink: 0;
-  border-radius: var(--radius-pill);
-  box-shadow: inset 0 0 0 1px color-mix(in srgb, var(--craft, var(--ink)) 40%, transparent);
-}
-.craft-review > .craft-tool {
-  box-shadow: none;
-  border-radius: var(--radius-pill) 0 0 var(--radius-pill);
-}
-.craft-review__more {
-  display: inline-flex;
-  align-items: center;
-  justify-content: center;
-  width: 24px;
-  height: 32px;
-  padding: 0 4px 0 0;
-  border: 0;
-  border-radius: 0 var(--radius-pill) var(--radius-pill) 0;
-  background: none;
-  color: var(--craft, var(--ink-soft));
-  cursor: pointer;
-}
-.craft-review__more svg { width: 12px; height: 12px; transition: transform 200ms cubic-bezier(.2, .7, .3, 1); }
-.craft-review__more[aria-expanded="true"] svg { transform: rotate(180deg); }
-.craft-review__more:hover { background: color-mix(in srgb, var(--craft, var(--ink)) 12%, transparent); }
-.craft-review__more:focus-visible { outline: 2px solid var(--focus); outline-offset: 2px; }
-@media (min-width: 1181px) {
-  .craft-review > .craft-tool { padding-right: 6px; }
-}
-@media (max-width: 1180px) {
-  .craft-review > .craft-tool { width: 30px; }
-}
-
-/* Reviewing: the pill takes a wash of the experience's colour, says
-   "Reviewing", and a drop of wet paint hangs from it. */
-.craft-tool.craft-tool--reviewing { position: relative; background: color-mix(in srgb, var(--craft, var(--sea)) 14%, transparent); }
-.craft-tool.craft-tool--reviewing::after {
-  content: '';
-  position: absolute;
-  left: 13px;
-  bottom: -4px;
-  width: 4px;
-  height: 7px;
-  border-radius: 0 0 3px 3px;
-  background: var(--craft, var(--sea));
-}
-
-/* ---------- the menu ---------- */
-.craft-review__menu {
-  position: absolute;
-  top: calc(100% + 8px);
-  right: 0;
-  z-index: var(--z-drawer);
-  display: flex;
-  flex-direction: column;
-  gap: 2px;
-  min-width: 248px;
-  padding: 6px;
-  box-sizing: border-box;
-  background: var(--bg-card);
-  border-radius: var(--radius);
-  box-shadow: 0 0 0 1px var(--glass-edge), var(--shadow-pop);
-}
-.craft-review__menu[hidden] { display: none; }
-/* A bridge over the gap, so a pointer travelling down to the menu never
-   leaves the pill and the hover menu does not close under it. */
-.craft-review__menu::before { content: ''; position: absolute; left: 0; right: 0; top: -8px; height: 8px; }
-.craft-review__head {
-  margin: 8px 12px 2px;
-  font-family: var(--label-face);
-  font-size: 11px;
-  letter-spacing: .12em;
-  text-transform: uppercase;
-  color: var(--ink-faint);
-}
-.craft-review__item {
-  display: flex;
-  align-items: center;
-  gap: 10px;
-  min-height: 36px;
-  padding: 0 12px;
-  border: 0;
-  border-radius: var(--radius-sm);
-  background: none;
-  color: var(--ink);
-  font: inherit;
-  font-size: 14px;
-  text-align: left;
-  white-space: nowrap;
-  cursor: pointer;
-}
-.craft-review__item:hover { background: color-mix(in srgb, var(--craft, var(--ink)) 10%, transparent); }
-.craft-review__item:focus-visible { outline: 2px solid var(--focus); outline-offset: -2px; }
-.craft-review__icon { width: 16px; height: 16px; flex-shrink: 0; color: var(--ink-soft); }
-.craft-review__item[aria-checked="true"] { color: var(--craft, var(--ink)); font-weight: 500; }
-.craft-review__item[aria-checked="true"] .craft-review__icon { color: var(--craft, var(--ink)); }
-.craft-review__check { width: 14px; height: 14px; margin-left: auto; flex-shrink: 0; }
-.craft-review__item--stop { margin-top: 4px; border-top: 1px solid var(--rule-soft); border-radius: 0 0 var(--radius-sm) var(--radius-sm); }
-
-/* A phone: no hover and no room for the chevron, so Review itself opens the
-   menu (craft-mode.js), and the menu spans the bar instead of hanging off a
-   pill that may sit near the left edge. */
-@media (max-width: 640px) {
-  .craft-review { position: static; }
-  .craft-review__more { display: none; }
-  .craft-review > .craft-tool { width: 28px; border-radius: var(--radius-pill); }
-  .craft-review__menu { top: calc(100% + 6px); left: 12px; right: 12px; min-width: 0; }
-  .craft-review__item { min-height: 44px; }
-}
 /* Under 360px the bar holds two tools, not three: reviewing Ideating, its
    own Ideas button folds away (craft-mode.js marks it a guest). */
 @media (max-width: 359px) { .craft-tool.craft-tool--guest { display: none; } }
 
 /* ---------- the scribble wall ---------- */
 /* The top bar's empty space, between the title and the controls. Where none
-   is left (a phone), the Review pill still says "Reviewing". */
+   is left (a phone), the review panel's mode row still says which. */
 .craft-graffiti {
   position: relative;
   flex: 1 1 0;