Skip to content

fix(ui): nav, banner, buttons, and overflow - #98

Merged
danyalprout merged 11 commits into
mainfrom
max/fixes
Aug 26, 2026
Merged

fix(ui): nav, banner, buttons, and overflow#98
danyalprout merged 11 commits into
mainfrom
max/fixes

Conversation

@barvian

@barvian barvian commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Sidebar Back now stays on the current page and keeps the drawer open on section roots. The active pill no longer slides when the banner or theme rerenders.
  • The global banner collapses on dismiss, and respects reduced motion.
  • The activity drawer no longer creates a horizontal scrollbar or extra page scroll. It sizes from the content column instead of 100vw, and overscroll stays inside the list.
  • Default buttons are 40px / 14px. The 16px input floor is iOS-only so desktop fields stay small. Upgrades cards wrap at mid widths. Spending-account addresses use CopyableValue.

Test plan

  • Open a Vibenet child page and tap Back in the sidebar. The root list slides in, the current page stays highlighted, and the mobile drawer stays open.
  • Dismiss the EIP-8130 banner. It collapses and the nav pill does not slide.
  • Toggle theme. The nav pill does not slide.
  • Account demo: no extra scroll below the activity drawer, and no horizontal scrollbar with the drawer open.
  • Scroll the activity list to the end. Overscroll does not pull the page.
  • Upgrades cards: the date/status row wraps instead of stacking at mid widths.
  • Token address / policy inputs: the button height matches the field.
  • Desktop inputs keep their small type. iOS Safari still does not zoom on focus.

Made with Cursor

@cb-heimdall

cb-heimdall commented Aug 26, 2026

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 26, 2026

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 26, 2026 12:31am

Request Review

@danyalprout
danyalprout merged commit c72c175 into main Aug 26, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants