Skip to content

feat(inbox): six more pieces for the thread panel's art - #2496

Draft
backnotprop wants to merge 1 commit into
feat/inbox-thread-panel-viewfrom
feat/inbox-panel-art-nine
Draft

backnotprop wants to merge 1 commit into
feat/inbox-thread-panel-viewfrom
feat/inbox-panel-art-nine

Conversation

@backnotprop

Copy link
Copy Markdown
Owner

Adds six pieces to the empty thread panel's rotation, which takes it from 15 to 21, and sets the owner's new order. The Inbox now opens on Red Line, with Keycap second.

What changes

  • New pieces: redline, bridgeplan, flowchart, clipboard, goggles, totman. Each has a light and a dark mask in packages/inbox/components/panel-art/, repacked losslessly to 1-bit palette PNGs.
  • The order is spread out on the owner's rule, so the helmets and faces do not sit together, and neither do the plans.
  • PIECES in EmptyThreadPanel.tsx is now the one place that says which pieces there are, their order and which comes first. The tests read that list or follow the rotation, so changing it needs no other edit.

What does not change

  • The panel view itself, and the default wide view.
  • packages/ui.

How it was tested

  • packages/inbox/threadPanel.test.tsx (DOM)
  • tests/e2e/inbox-thread-panel.spec.ts (Playwright, against the compiled binary), which now walks the rotation until it returns to the first piece.

The base is feat/inbox-thread-panel-view, #2479. This PR moves to main once that merges.

…ed Line

Adds redline, bridgeplan, flowchart, clipboard, goggles and totman, 21 pieces in all, in the owner's order. PIECES is now the one place for the set, the order and the first piece: the tests read it or follow the rotation, so neither the first piece nor the count is pinned anywhere else.
backnotprop added a commit that referenced this pull request Nov 2, 2026
The owner's call: the panel shows only "No thread selected", centered in the muted color. The masks, the rotation button, its state in App.tsx, the forced-colors rule and the art's tests are gone; the art waits in #2496. The option, its cookie and the layout's guards are unchanged.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant