Skip to content

fix: make WalletModal responsive with touch-friendly UI and mobile bottom-sheet (closes #136)#233

Open
Emmanard wants to merge 1 commit into
kellymusk:mainfrom
Emmanard:fix/issue-136-wallet-modal-responsive
Open

fix: make WalletModal responsive with touch-friendly UI and mobile bottom-sheet (closes #136)#233
Emmanard wants to merge 1 commit into
kellymusk:mainfrom
Emmanard:fix/issue-136-wallet-modal-responsive

Conversation

@Emmanard

Copy link
Copy Markdown
Contributor

Summary

WalletModal now renders as a vaul bottom-sheet on mobile and keeps the existing Dialog on sm: and above. All four inner views are untouched.

Changes

  • hooks/use-media-query.ts — new SSR-safe hook wrapping window.matchMedia
  • components/Wallet/WalletModal.tsx — mobile Drawer branch added; Desktop Dialog branch unchanged

Approach

  • Uses vaul (already installed at ^1.1.2) via its native API, same pattern as navbar.tsx
  • Zero new libraries introduced
  • All inner views (InstallView, ConnectView, ConnectingView, NetworkWarningView) byte-for-byte untouched
  • Existing CTAs already meet 44px minimum tap target (h-12)
  • Bottom-sheet uses max-h-[90svh] to correctly account for mobile browser address bar
  • rounded-t-[1.25rem] matches --radius token from globals.css

Closes #136

@vercel

vercel Bot commented Jun 27, 2026

Copy link
Copy Markdown

@Emmanard is attempting to deploy a commit to the kelly musk's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jun 27, 2026

Copy link
Copy Markdown

@Emmanard Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

17. Mobile-responsive wallet modals

1 participant