LAYOUT-2: fix top-bar control overlap at ~1024px#107
Merged
Conversation
The three top-bar sections (graph selector / 8 view-mode buttons / status) overflowed at 1024px and the centered button strip rendered on top of the graph selector text (audit OVERLAP: "Graph View" × selector, 100%). Constrain the layout so sections never collide: - center strip is flex-1 min-w-0 and scrolls within its slot; - it only centers at xl (>=1280, where it fits) — justify-start at lg so its left edge stays in its slot instead of being pushed under the selector; - left selector is compact (md:max-w-xs md:shrink); right status md:shrink-0. Verified via the audit's overlap scan: 0 overlaps @1024. THE GATE 5/5. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…lear of header + panel The toggle was `fixed right-4 top-20` (viewport-relative) while the panel it opens is absolute within the content container — so it overlapped the global header on desktop (user report). Make it `absolute top-4` within the same container as the panel, and shift it left of the 320px panel when open so it stays clickable. Clear, collision-free positioning. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
🧪 Comprehensive Test Suite
Full-stack smoke gate runs in the CI workflow. |
1 similar comment
🧪 Comprehensive Test Suite
Full-stack smoke gate runs in the CI workflow. |
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 audit's new overlap scan caught it: at 1024px the centered 8-button view strip rendered on top of the graph-selector text (
"Graph View" × selector, 100% overlap). Constrained the 3-section top bar so they never collide — center strip isflex-1 min-w-0and scrolls within its slot; centers only atxl(≥1280, where it fits),justify-startatlg; left selector compact, right statusshrink-0.Verified via the audit overlap scan: 0 overlaps @1024. THE GATE 5/5.
🤖 Generated with Claude Code