Skip to content

feat: rename Connect MetaMask to generic Connect Wallet#21

Merged
dzianisv merged 1 commit intomainfrom
feat/generic-evm-wallet
Feb 16, 2026
Merged

feat: rename Connect MetaMask to generic Connect Wallet#21
dzianisv merged 1 commit intomainfrom
feat/generic-evm-wallet

Conversation

@dzianisv
Copy link
Copy Markdown
Contributor

Summary

  • Renames "Connect MetaMask" button to "Connect Wallet" so it works with any EIP-1193 EVM wallet (MetaMask, Rabby, Coinbase Wallet, Rainbow, Brave Wallet, etc.)
  • Updates install prompt from "MetaMask not detected" to "EVM wallet not detected"
  • Changes error messages to be wallet-agnostic
  • Replaces MetaMask orange brand color with neutral blue (#3b82f6)
  • No API/backend changes

Changes

File Change
web/src/components/wallet-connect.tsx Button text, install prompt name, button color
web/src/lib/wallets/evm.ts Error messages, file comment
web/src/lib/config.ts Comment update
web/src/__tests__/components/wallet-connect.test.tsx All 13 tests updated to match new labels

Testing

  • pnpm build passes
  • 57/57 web tests pass

- Rename button text from 'Connect MetaMask' to 'Connect Wallet'
- Update install prompt from 'MetaMask not detected' to 'EVM wallet not detected'
- Change error messages to generic ('No EVM wallet detected', 'add network to your wallet')
- Replace MetaMask orange (#f6851b) with neutral blue (#3b82f6)
- Update all 13 wallet-connect tests to match new labels
- Keep metamask.io/download as the install URL (still the best EVM wallet install page)
- No API changes — works with any EIP-1193 wallet (MetaMask, Rabby, Coinbase, etc.)
@github-actions
Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-pebble-0dd69340f-21.eastus2.4.azurestaticapps.net

@dzianisv dzianisv merged commit 5547291 into main Feb 16, 2026
2 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.

1 participant