Repository navigation
Stop the Explorer label overlapping the File button in narrow sidebars - #21
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: 0434f441-846f-4ac5-8ca7-1d79c8231498) |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
d26a7dd to
1a50067
Compare
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: 71c98e30-8833-4b93-8a95-505298ec3cf3) |
|
claude-opus-5-5 RESPONDING ON BEHALF OF The1UNeed Merging despite two non-green items, neither caused by this diff:
|
When the workspace sidebar is dragged narrow (below ~212px; the minimum is 180px), the "Explorer" heading ran underneath the "+ File" button and the two collided.
Now the header keeps its normal look at the default width and above, and below 224px the File/Folder buttons drop their text and show icons only (they already have
aria-labelandtitle). The heading also truncates instead of overflowing, so longer translations can't collide with the buttons either.Proof: headless Playwright against
next dev, measuring the heading's right edge vs. the File button's left edge at sidebar widths 180/200/220/240/260/480. Before: overlap at 180 and 200 (heading ends at x=67, button starts at x=36/56). After: no overlap at any width, and at ≥240px button positions are identical to before.pnpm typecheck,pnpm lint,pnpm test:web(224 passed) all green.Reviewed with sol-loop: 1 round with gpt-6.1-sol. Sol verdict: approved
Generated by
claude-opus-5-5using theClaude Code (T3 Code)harness.🤖 Generated with Claude Code
Note
Low Risk
CSS-only header layout tweaks in the file explorer sidebar with no logic or data-handling changes.
Overview
Fixes a layout collision when the workspace sidebar is resized very narrow: the Explorer title could draw under the + File / + Folder controls.
The sidebar header now uses a container query (
@container) so that below ~12rem width the File and Folder actions keep icons only (labels hidden via@max-[12rem]:hidden;aria-label/titleunchanged). The Explorer label gets truncation (max-w-full truncate), and the action group isshrink-0so the buttons stay intact while the title yields space.Reviewed by Cursor Bugbot for commit 1a50067. Bugbot is set up for automated code reviews on this repo. Configure here.