-
Notifications
You must be signed in to change notification settings - Fork 10
Test #1527
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test #1527
Conversation
- Sidebar starts collapsed (56px) and expands on hover (240px) - Smooth transitions: icons stay in place, text fades with max-w + opacity - Logo shows icon when collapsed, icon + 'Recoupable' text when expanded - Nav buttons use icon-only centered squares when collapsed - Active page accent bar in brand color (#345A5D) - Compact chat items (text-xs, tighter padding) - User profile: compact avatar at bottom, always visible - Moved 'Recoupable Pro: Active' badge into account dropdown - New Chat uses SquarePen icon instead of plus - Uses Plus Jakarta Sans for brand name - Light/dark mode logo support (icon-lightmode/darkmode SVGs) - Removed MiniMenu dependency (no longer used)
- Consistent font weight (font-normal) across all nav and chat items - Stroke-based robot icon replacing filled version - Chat items same font size as nav (text-sm) - Tighter chat item spacing (py-[6px]) - Section label: lowercase 'Chats', removed divider line - Chat items aligned with nav (px-3) - Reduced sidebar widths (240px expanded, 56px collapsed) - Brand name reduced to text-base - Active accent bar in brand color - Removed extra spacer between New Chat and nav items
feat: sidebar expand on hover with polished UI
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThe changes introduce a hover-based sidebar expansion mechanism by adding an Changes
Sequence DiagramsequenceDiagram
participant User
participant Sidebar
participant Menu
participant NavButton
participant Logo
participant UserInfo
User->>Sidebar: onMouseEnter
Sidebar->>Sidebar: setIsExpanded(true)
Sidebar->>Menu: isExpanded=true
Menu->>Logo: isExpanded=true
Logo->>Logo: Show brand name (opacity transition)
Menu->>NavButton: isExpanded=true
NavButton->>NavButton: size="sm", show label, add accent bar
Menu->>UserInfo: isExpanded=true
UserInfo->>UserInfo: Forward to UserProfileButton
UserInfo->>UserInfo: Show name & chevron (opacity transition)
Menu->>Menu: Render RecentChats (AnimatePresence)
Menu->>Menu: Show full chat list with fade-in
User->>Sidebar: onMouseLeave
Sidebar->>Sidebar: setIsExpanded(false)
Sidebar->>Menu: isExpanded=false
Menu->>Logo: isExpanded=false
Logo->>Logo: Hide brand name (opacity transition)
Menu->>NavButton: isExpanded=false
NavButton->>NavButton: size="icon", hide label, remove accent bar
Menu->>UserInfo: isExpanded=false
UserInfo->>UserInfo: Hide name & chevron (opacity transition)
Menu->>Menu: Hide RecentChats (AnimatePresence fade-out)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes The changes introduce interconnected prop propagation across multiple sidebar components with new animation logic and mouse event handling. While individual nav item updates are repetitive (reducing review complexity), the core Menu and NavButton components contain significant logic restructuring. The pattern is consistent and well-scoped, but the breadth of file modifications and animation integration warrant careful verification of prop flow and styling transitions. Possibly related PRs
Suggested reviewers
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary by CodeRabbit
Release Notes
New Features
UI & Styling