Repository navigation
feat(source-control): edge-anchored sidebar toggle, branches header cleanup, orange current-branch accents - #349
Conversation
…leanup, orange current-branch accents
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (3)
Included review availability: 3 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughThe PR updates sidebar collapse controls, branch controls, responsive settings interfaces, and prompt-overlay sizing. It adds accessibility attributes, collapsible configuration content, responsive layouts, and component tests while preserving existing settings and source-control actions. ChangesSidebar controls
Branch interface
Settings interface
Session overlay handling
Priority: ➖ Normal Estimated code review effort: 4 (Complex) | ~60 minutes Merge Risk: ⚪ Minimal · up to The update refreshes source-control and settings UI behavior while ensuring prompt-overlay spacing is restored after edit remounts. No current merge-blocking risk is identified. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Comment |
…d mobile menu landing
…le stop button above input
…leanup, orange current-branch accents (chriswritescode-dev#349) * feat(source-control): edge-anchored sidebar toggle, branches header cleanup, orange current-branch accents * feat(settings): sidebar navigation with per-tab two-column layouts and mobile menu landing * fix(ui): re-measure prompt overlay after edit remount and anchor mobile stop button above input
Problem
The source-control panel had a bulky sidebar toggle in the desktop sidebar header and a cluttered branches tab: a redundant "Current: " header, a detached search input, and full-width buttons. Current-branch accents also reused the same emerald used for ahead/stage states, so the checked-out branch didn't stand out.
Fix
Sidebarrenders its own toggle (only whenonToggleis passed) as an absolute vertical pill on the right edge, with an orange accent bar, hover chevron, andaria-expanded; removed the toggle row fromDesktopSidebarh-10bg-orange-500/10andGIT_UI_COLORS.currentchanged from emerald to orange, distinguishing "current" from ahead/stage statesaria-expandedbehaviorTesting
vitest src/components/ui/sidebar.test.tsx: 11 passedpnpm lint:frontendcleantsc -b --noEmitcleanHow to test manually
Open the source-control panel: hover the sidebar's right edge for the collapse pill, check the Branches tab header (search + compact Branch/Worktree buttons), and note the orange accent on the current branch row and panel header.
Summary by CodeRabbit