Skip to content

Conversation

Ikari-Shinji-re
Copy link
Member

@Ikari-Shinji-re Ikari-Shinji-re commented Aug 6, 2025

Summary

This change is similar to the pull request: #1224

Destination Wallet Page
The destination wallet page allows users to either select/connect their preferred wallet or enter/paste a custom recipient address.

Wallet Selection

  • Show only wallets compatible with the selected blockchain (e.g., MetaMask or WalletConnect for Ethereum, Phantom for Solana).
  • For each wallet option, display its name and icon.

Custom Destination Input

  • Provide an input field below the wallet list for entering an address.

  • Support the following UI states:

    • Empty: placeholder text prompting the user to enter or paste an address.
    • Edit: when typing or pasting, show the editable text with normal formatting.
    • Error: if validation fails, highlight with a red outline and display an inline error message.
    • Disabled: when the user has already provided an address.
  • Add two action buttons beside the input:

    • Paste: inserts text from the clipboard.
    • Clear: erases the current input.

Fixes # (issue)

How did you test this change?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • Implemented a user interface (UI) change, referencing our Figma design to ensure pixel-perfect precision.

Copy link
Collaborator

@yeager-eren yeager-eren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just checked the store logics. lgtm.

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