Show the comment round-trip in the README - #110
Merged
Merged
Conversation
The README opened with a single still, which shows the diff but not the part that makes a review a review. `docs/comment-on-a-line.gif` runs the whole round-trip: a line in the diff is hovered, the composer opens, a question is typed and submitted, and Claude Code answers in the same thread. The frames are the `comment-and-reply` beat captured for the ad by `scripts/ad/capture-ad.mjs` on `worktree-ad-video` — no new recording, so the UI matches the existing still. Cut to 1600x900 at 20fps with a 1.4s hold on the last frame so the reply is readable before the loop restarts; 2.2 MB, displayed at 800 like the still beside it. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The README was split into a front page and eight guides while this branch was open, which removed the paragraph the GIF was anchored to. Took main's README whole and re-placed the GIF at the end of "What it does", where it now follows the "agents are first-class" bullet rather than the old cache paragraph. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The README opened with one still. It shows the diff, but not the part that makes a review a review — so a first-time reader never sees the app move.
docs/comment-on-a-line.gifcloses the What it does section, right after the "agents are first-class" bullet, and runs the whole round-trip: a line in the diff is hovered, the blue+appears, the composer opens, a question about Escape handling is typed and submitted, and Claude Code answers in the same thread. The lead-in reads:Where the frames came from. The
comment-and-replybeat captured for the ad byscripts/ad/capture-ad.mjsonworktree-ad-video— nothing was re-recorded, so the UI matches the still already at the top of the README.Encoding. 1600×900 at 20fps, displayed at
width="800"like the still, so it is 2× on a retina screen. A 1.4s hold on the last frame keeps the reply readable before the loop restarts. 2.2 MB, palette of 128 colours withstats_mode=diffanddiff_mode=rectangle— the dark, mostly static UI compresses well, so full resolution costs less than a downscale at a naive palette would.Note on the merge commit. This branch was opened before #109 split the README into a front page and eight guides, which removed the paragraph the GIF was anchored to. Main was merged in, its README taken whole, and the GIF re-placed in the new structure. The diff against main is still only those six lines plus the asset.
🤖 Generated with Claude Code