Skip to content

fix(ui): sidebar back-slide, overflow - #91

Merged
barvian merged 5 commits into
mainfrom
cursor/fix-sidebar-back-slide-direction
Aug 25, 2026
Merged

fix(ui): sidebar back-slide, overflow#91
barvian merged 5 commits into
mainfrom
cursor/fix-sidebar-back-slide-direction

Conversation

@barvian

@barvian barvian commented Aug 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Fix the section back-header so the first click slides the pane backward instead of reusing the inbound direction.
  • Put the Base logo in the shared sidebar scroller so it rolls away with the list, and drop the top fade (bottom fade only).
  • Use Base UI ScrollArea for a thin custom thumb in the nav
  • Consolidate copy value animation

Test plan

  • Click into a section (e.g. Vibenet), then the back header: the pane should slide back, not keep the inbound direction.
  • Scroll the sidebar: the logo leaves with the list, no fade at the top, bottom fade still appears when there is overflow.
  • Open a section while already scrolled: the logo should return at the top.
  • Mobile drawer still has no logo in the scroller.
  • Copyable values still copy.

Made with Cursor

@cb-heimdall

cb-heimdall commented Aug 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews ✅ 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@vercel

vercel Bot commented Aug 25, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
omni-ui Ready Ready Preview Aug 25, 2026 8:57pm

Request Review

@barvian barvian changed the title fix(ui): sidebar back-slide, scroller, and scrolling logo fix(ui): sidebar back-slide, overflow Aug 25, 2026
@barvian
barvian requested a review from montycheese August 25, 2026 20:32
barvian and others added 4 commits August 25, 2026 13:48
@barvian @claude @cursoragent
fix(ui): slide the sidebar back the right way on first header click
710be2d 
Direction was applied in an effect, so AnimatePresence still used the inbound value on the first back-navigation.

Co-Authored-By: Claude <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@barvian
fixes
043ae24
@barvian
include base logo in scrollable area
7f62926
@barvian @cursoragent
fix: point new lockfile tarballs at the public npm registry
a638ad8 
GitHub Actions cannot resolve registry-npm.cbhq.net, so npm ci failed on the four packages added with @base-ui/react.

Co-authored-by: Cursor <cursoragent@cursor.com>
@barvian
barvian force-pushed the cursor/fix-sidebar-back-slide-direction branch from f221ed1 to a638ad8 Compare August 25, 2026 20:48
@barvian
barvian merged commit 05e6cf4 into main Aug 25, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danyalprout danyalprout danyalprout approved these changes
@montycheese montycheese Awaiting requested review from montycheese

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@barvian @cb-heimdall @danyalprout