Skip to content

The composer's top lists every change that waits for review - #56

Merged
fxck merged 2 commits into
mainfrom
fix/strip-every-change
Oct 1, 2026
Merged

fxck merged 2 commits into
mainfrom
fix/strip-every-change

Conversation

@fxck

@fxck fxck commented Oct 1, 2026

Copy link
Copy Markdown
Member

When a Mate had two changes waiting, the composer's top strip showed only one: "… waiting for your review of #1", with one title and one Review.

Now:

  • One change waiting: the strip reads as before.
  • Two or three: the strip reads "… waiting for your review of 2 changes". It then shows one line per change, newest first, each named by the repository rule and each with its own Review.
  • Four or more: the newest three, then "and N more", which opens the project's changes.

Each line's Review is the menu's blue word rather than a button, so blue buttons never stack. The lines are remembered with the strip, so a reload doesn't shift the composer.

Tests:

  • a mateNextStep table for one, two, three and five changes (RED → GREEN);
  • a two-change render where each Review opens its own change;
  • reload memory with the lines.

🤖 Generated with Claude Code

fxck added 2 commits October 1, 2026 12:23
It showed only the newest. Where two or three wait, it says how many and
lists each, newest first and named as the menu names it, with its own
Review; past three, the newest three and "and N more", which opens the
project's page. One waiting reads as before. The lines are remembered
with the strip, so a reload paints the strip it keeps.
@fxck
fxck merged commit 63d0c56 into main Oct 1, 2026
11 checks passed
fxck added a commit that referenced this pull request Oct 1, 2026
- #50 A person's own new project never waits on the path for someone else's
- #51 A docked operation offers to open only when opening adds something
- #52 A change's review reads the change itself, and rows name their repository when a Mate has two
- #54 A coming-up Mate's page hands over as soon as the Mate answers, whatever the browser kept
- #53 The account's line: Try now shows it tries, names what isn't answering, and one org's trouble holds no other
- #56 The composer's top lists every change that waits for review
- #55 A message echoed in the run card shows its words and its pictures
- #57 One stalled subscription retries alone instead of replacing the organization's socket
- #58 Docs: what the hours after 0.11.74 measured, and the fixes they took
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