Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Send Fund UX #64

Closed
lw-cdm opened this issue Apr 26, 2024 · 2 comments · Fixed by #116
Closed

Update Send Fund UX #64

lw-cdm opened this issue Apr 26, 2024 · 2 comments · Fixed by #116
Assignees

Comments

@lw-cdm
Copy link

lw-cdm commented Apr 26, 2024

  • Move token selector to top
  • Account selector (Send from) is based on token selector value
  • Address book in address input (Send to) must be filtered based on token selector and Send from values
@lw-cdm lw-cdm self-assigned this Apr 26, 2024
@lw-cdm lw-cdm assigned dungnguyen-art and unassigned lw-cdm May 27, 2024
@dungnguyen-art dungnguyen-art linked a pull request May 30, 2024 that will close this issue
@haiyenvu23
Copy link

haiyenvu23 commented May 31, 2024

Version test : http://14.224.129.242:8900/s/ds8DpRyWMt5z4gC

  • Update UI for Send from field and Send to field

  • Error when input EVM address on Send to field
    image

  • Hide Estimated fee display information when sending EVM tokens on Send funds screen
    image

  • Input data is reset when changing token
    Actual :
    1122

Expect : When changing the token, the account address must also change

  • Hide this context
    image

@haiyenvu23
Copy link

haiyenvu23 commented May 31, 2024

Version test : http://14.224.129.242:8900/s/PcfQSbB87bXyNHd

  • Put the icon on top

Image

  • Hide this icon

Image

  • Change text : "Send from" -> "From" , "Send to" -> "To"

Image

  • UI

Image

  • Remove this line

Image

  • Account address is reset when changing token

Image

  • UI

Image

  • UI bug when entering an address outside the wallet

Image

  • Change size icon = 16px , edit padding for icon

Image

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 a pull request may close this issue.

4 participants