Skip to content

Enhance layout: sidebar and mobile menu#168

Merged
QSchlegel merged 1 commit intomainfrom
Sidebar-Layout
Nov 10, 2025
Merged

Enhance layout: sidebar and mobile menu#168
QSchlegel merged 1 commit intomainfrom
Sidebar-Layout

Conversation

@christiangruender
Copy link
Contributor

@christiangruender christiangruender commented Nov 9, 2025

Summary

This PR improves the layout system with a sidebar navigation that is only visible when needed and an improved mobile experience.

Sidebar Concept

  • Focused sidebar on wallet context (removed general navigation links)
  • Sidebar now only appears in wallet-specific views

Sidebar UI

  • Enhanced menu organization with section header
  • Moved wallet dropdown selector to sidebar
  • Added "New Wallet" action directly in dropdown
  • Added text truncation for long wallet names
  • Standardized icon sizes
  • Consistent text sizing

Mobile Navigation

  • Custom overlay implementation
  • Improved interaction handling
  • Smooth slide-in/out animations

Header

  • Optimized logo size for better visual balance
  • Simplified header structure

Theme Support

  • Enhanced dark/light mode support (clearer layout lines in light mode)

Changed Files

  1. layout.tsx - 148 lines changed (+34, -114)
  2. multisig-wallet.tsx - 159 lines changed (+115, -44)
  3. mobile-navigation.tsx - 153 lines changed (+88, -65)
  4. wallet-nav-link.tsx - 30 lines changed (+13, -17)
  5. sheet.tsx - 5 lines changed (+3, -2)
  6. logo.tsx - 2 lines changed (+1, -1)
  7. menu-link.tsx - 2 lines changed (+1, -1)
  8. dropdown-menu.tsx - 2 lines changed (+1, -1)

@vercel
Copy link

vercel bot commented Nov 9, 2025

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

Project Deployment Preview Comments Updated (UTC)
multisig Ready Ready Preview Comment Nov 9, 2025 9:46pm

@QSchlegel
Copy link
Collaborator

Looks good.

@QSchlegel QSchlegel merged commit d46752f into main Nov 10, 2025
4 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.

2 participants