Skip to content

Issue 046: Wire WalletSelectModal into App connect flow for multi-wallet UX #841

Description

@orunganiekan

Category: Frontend
Complexity: 200 points
Labels: frontend, wave-200

Context

WalletSelectModal.tsx and adapters (Freighter, xBull, Lobstr, Hana) exist via useWallet, but App.tsx still hardcodes connect(AVAILABLE_WALLETS[0]) and Freighter-only copy.

Problem

Users with non-Freighter wallets cannot discover supported wallets despite completed adapter work.

Goal

Integrate modal + WalletBar patterns into the primary connect path with accessible selection and error states.

Scope

Includes: App/connect UX wiring, tests.
Does NOT include: new wallet adapters.

Implementation Guidelines

  • Files: App.tsx, WalletSelectModal.tsx, WalletBar.tsx, useWallet.ts, App.test.tsx.
  • Preserve passphrase/network checks from useNetworkCheck.

Acceptance Criteria

  • Connect opens wallet selector
  • Each installed adapter selectable in UI tests/mocks
  • Freighter-only copy removed
  • Keyboard/focus trap covered via existing hooks if used

Validation

cd frontend && npm test for App/Wallet tests; manual smoke with mocked adapters.

PR Expectations

The PR should include:

  • UI wiring + tests + screenshot optional


Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Stellar WaveIssues in the Stellar wave programfrontendwave-200200-point contribution wave issue

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions