Skip to content

Feat/wallet - #85

Merged
Nabeelahh merged 2 commits into
Vaulty-X:mainfrom
ummarig:feat/wallet
Jul 27, 2026
Merged

Feat/wallet#85
Nabeelahh merged 2 commits into
Vaulty-X:mainfrom
ummarig:feat/wallet

Conversation

@ummarig

@ummarig ummarig commented Jul 27, 2026

Copy link
Copy Markdown

Changes Made:

  1. Created new WalletAccountMenu.tsx - A comprehensive, accessible dropdown menu component that:

    • Shows connected/disconnected state
    • Displays shortened public key and current network (testnet/mainnet)
    • Includes connect, disconnect, and error state handling
    • Supports full keyboard navigation and screen reader accessibility
    • Clicks outside to close, ESC key to close, and proper focus management
    • Responsive design that works on all screen sizes
  2. Updated layout.tsx - Replaced the old WalletButton with the new WalletAccountMenu, ensuring the wallet connection state is visible on every page.

Key Features Implemented:

  • Consistent state across all pages: The header now shows wallet status everywhere
  • Shortened public key: Displays GABC…WXYZ format while showing full key on hover
  • Network indicator: Shows which network (testnet/mainnet) the wallet is connected to
  • Connect/Disconnect actions: Proper UI for both states with loading states
  • Error handling: Maintains all existing error states with proper accessibility roles
  • Keyboard accessible: Full support for keyboard navigation (Enter, Space, Arrow keys, ESC, Tab)
  • No sensitive data: Never displays private keys or transaction secrets
  • Responsive design: Works on mobile and desktop with proper Tailwind styling

The implementation maintains compatibility with all existing functionality while providing a much improved user experience for wallet connection management.

closes #80

@Nabeelahh
Nabeelahh merged commit 03dcf11 into Vaulty-X:main Jul 27, 2026
1 check 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.

[Frontend] Add a persistent wallet account header

2 participants