Public: anyone can read this; the crew can change it.

"Look" means three things, two of them on the same page #176

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

Pushed successfully.

shipped production is serving it deploy d5b2fe2

opened this

Pushed successfully.

The studio's repo-header tab to /changes said "Look" — the same word the five-word table uses for a green pull ready for review, and the same word on the pull's own picture card. Now it says "Changes," matching the noun /changes and "Every change" already use elsewhere in the studio, and its icon changed from the eye (which was quietly doing the same disambiguating job) to the pull icon dev's own Pulls tab already uses.

What changed:

  • src/web/chrome.ts: the tab's label and icon.
  • docs/STUDIO.md: the tab-strip mention updated, plus one new paragraph spelling out which of the three meanings "look," "change," and the picture card each own.
  • test/studio-words.test.ts: the app page's row now asserts the tab strip carries the pull-icon "Changes" link and never the eye-icon "Look."

This landed on top of #174 (already merged to main), which introduced src/web/studio/nouns.ts and touched the same lines — I rebased onto origin/main and reconciled the conflict there rather than diverging from it.

Left untouched, per the acceptance criteria: the picture card's own <h2>Look</h2> heading, the five-word look badge/state, ?word=look, and dev's Pulls tab.

bun test and tsc --noEmit are clean for everything this touches. Six unrelated tests fail in this sandbox on a clean origin/main too (headless-Chromium screenshot capture and a git spawn issue) — confirmed pre-existing, not caused by this change.

Closes #172

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs