Skip to content

Extension - Improve UI after Bitcoin integration #4316

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

Closed
6 tasks done
haiyenvu23 opened this issue Apr 25, 2025 · 4 comments · Fixed by #4384
Closed
6 tasks done

Extension - Improve UI after Bitcoin integration #4316

haiyenvu23 opened this issue Apr 25, 2025 · 4 comments · Fixed by #4384
Assignees
Milestone

Comments

@haiyenvu23
Copy link

haiyenvu23 commented Apr 25, 2025

Describe the bug

@Sokol142196 Sokol142196 changed the title Extension - Handle synchronized display of avatars for accounts on screens Extension - Some update UI related to Bitcoin integration Apr 28, 2025
@Sokol142196 Sokol142196 added this to the Milestone 9 milestone Apr 28, 2025
@dungnguyen-art dungnguyen-art changed the title Extension - Some update UI related to Bitcoin integration Extension - Improve UI after Bitcoin integration May 20, 2025
@haiyenvu23
Copy link
Author

haiyenvu23 commented May 23, 2025

Version test: https://file.cdmteck.com/s/Zwb5pWm6ymj6NtZ

  • Re-check padding for block on Account name field
    Actual:

    Image

    Expect:

    Image

  • UI bug in case address not in wallet
    Actual:

    Image

    Image

    Image

    Expect:

    Image

  • Update logo for Taproot address
    Actual:

    Image

    Expect: Show bitcoin logo

  • Sort list address type ( mainnet and testnet)
    Actual:

    Image

    Expect: Sort in order: Native SegWit -> Taproot -> Legacy

  • Update UX when get address bitcoin on list address screen
    Actual:

    Image

    Image

    Expect: Handle like UX get address of bifrost polkadot
    Note: show Native Segwit address next to network name

    Image

  • Update content
    Actual:

    Image

    Expect: Remove the ending " BTC address"

  • Update content for tooltip on button next/previous
    Actual:

    Image

    Expect: Don't show tooltip

@haiyenvu23
Copy link
Author

haiyenvu23 commented May 27, 2025

Version test: https://file.cdmteck.com/s/BHPsFMSAD5BcGLW

@haiyenvu23
Copy link
Author

haiyenvu23 commented May 28, 2025

Version test: https://file.cdmteck.com/s/AiL467TTanKzyQ8

  • Update network name
    Actual:

    Image

    Expect: Display network name

  • Re-check size for ecosystem icon (no changes needed; the current size is 16px)
    Actual:

    Image

    Expect:

    Image

  • Re-check padding (already fixed; this was a previous bug.)
    Actual: 6px

    Image

    Expect: 4px

@haiyenvu23
Copy link
Author

haiyenvu23 commented May 29, 2025

Version test: https://file.cdmteck.com/s/wMnPpxZHmKGYzn4
Validation passed in case:

  • Install/Update version

  • Synchronized display of avatars for accounts on some screens

    Image

  • Update UI/UX on Receive address features(All account/Specific account mode)

    Image

    Image

  • Add additional type address to Bitcoin address

    Image

    Image

  • Add system's tag for Account in "Derive account" tab on Account Details screen to clarify which system Solo account belongs to

    Image

  • Update UX when get address bitcoin on list address screen

    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.

3 participants