Repository navigation
fix(ui): drop Diff header totals that don't fit instead of running under the toolbar - #596
Merged
Merged
Conversation
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.
Map ^684. The Diff header's Committed/Uncommitted/Untracked totals were
shrink-0inside amin-w-0row, so when the pane got narrow (details sidebar open at 1280px) they overflowed under the toolbar andUntracked +1 -0overlapped Rebase. The totals now sit in a one-line, overflow-hidden wrapping row after the lane name: a total that doesn't fit wraps out of sight as a whole, the lane name keeps its place, and the toolbar is never covered.Smoke: isolated host on port 4760, lane with committed, uncommitted and untracked work; headless Playwright at 1280 (details open), 1000, 860 and 760 px. All three totals show at 1280; at 860 Untracked drops; at 760 only Committed shows; nothing overlaps Commit or the view buttons.
Built in Ace
No. Built by Claude Code as a fix-train worker for the ace-map review (Playwright for the smoke screenshots), because the train runs outside an Ace channel. Dogfooding gap tracked in #5.