Repository navigation
Measure composer clearance beside the inline Ask section (C61) - #491
Merged
Merged
Conversation
…ection (C61) CR-6 built Ask inline rather than as the floating composer C61 was planned for, so clearance was never re-measured. The new test checks Ask and the last activity item at 900x600 and 1440x900, with the review panel shut and open. It fails under an overlay and under a panel that takes the pane's height. The layout needed no change. Also bring the CR-8 matrix up to date: #477 already closed C22 and D6, and authenticated GET-only read to Val's session. Refs #319, #323, #327. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
* Add worktree retire for unmerged work the owner gives up prune keeps unmerged work forever because it has no merge proof. retire replaces that proof with a verified snapshot: dirty tracked and untracked files become a commit on the tip (built in a temporary index), unknown ignored data becomes a 0600 ignored.tar, and both go through the existing bundle/restore/fsck archive. Open PRs, active processes, Git operations, locks, hidden index entries and a moved tip still preserve the work. Removal stays non-forced and the branch is deleted by expected SHA. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * docs(worktrees): record pushed retirement repair candidate --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Review of #490 found the C76 contrast check sampled only the first match of each selector. It now measures every match and the whole tree and diff regions, which found Refresh files, Retry and Cancel context at 1.72:1 to 2.29:1 in all three themes, because they took the browser's button colours. Also drop the C61 test's assertion that the window reaches the requested height. Hosted runners cap it at their screen (677 px), which failed #491. Refs #319, #327. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
# Conflicts: # docs/status.md
This branch was successfully deployed
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
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.
Stacked on #490, targeting
cr-v050. Draft: like #490, this waits for Val and shouldn't be enqueued.Closes the CR-8 matrix's agent follow-ups that #490 left open.
overview.locator, which closes C22 and ends D6. Verify Code Review activity opening and comment link controls #481 and Verify remote diff controls and divider persistence #484 cover C50, C51, C63, C66, C69, C71 and C75. Those rows still said unverified.Evidence, all synthetic
7e45440: build and unit exit 0; desktop E2E 497 passed, 7 skipped, 0 failed; site E2E 7 passed. The commits here differ from that SHA only indocs/, after re-stacking on Add a Code Review setting, with accessibility, budget and regression gates #490's results commit.Hosted CI is running on this PR; it had no result when this was written.
Not established
No live provider, live GitHub read, VoiceOver or reference comparison. Clearance was measured with the synthetic
claudeandgh, in the test's window sizes.🤖 Generated with Claude Code