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

Confusing UX on Brave Wallet Onboarding #26638

Closed
ianrandmckenzie opened this issue Nov 9, 2022 · 3 comments · Fixed by brave/brave-core#16644
Closed

Confusing UX on Brave Wallet Onboarding #26638

ianrandmckenzie opened this issue Nov 9, 2022 · 3 comments · Fixed by brave/brave-core#16644
Assignees
Labels
feature/web3/wallet Integrating Ethereum+ wallet support front-end-change This task is a front end task and doesn't need any C++ changes OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/include

Comments

@ianrandmckenzie
Copy link

ianrandmckenzie commented Nov 9, 2022

Description

On the page prompting the user to connect their wallet, there is a link that says, "Learn more about Brave Wallet"
brave://wallet/crypto/onboarding/welcome

When clicking the link, it takes you here:
https://brave.com/web3/what-is-crypto-wallet/

This article describes crypto wallets, but says nothing about Brave Wallet itself. For people that are already familiar with what crypto wallets are, it creates a frustrating disconnect in that they are likely wanting to know about the Brave Wallet, not crypto wallets, as the link text would suggest.

Steps to Reproduce

  1. Go to any website that has MetaMask integration and click a login with MetaMask link
  2. A new tab opens brave://wallet/crypto/onboarding/welcome
  3. Click the "Learn more about Brave Wallet" link at the bottom with the assumption that you are learning about Brave Wallet, not crypto wallets.

Actual result:

An article about crypto wallets.

Expected result:

An article about Brave Wallet.

Reproduces how often:

100%

Brave version (brave://version info)

1.45.118 Chromium: 107.0.5304.91 (Official Build) (arm64)

Version/Channel Information:

I'm sorry, I don't know what channels are.

Other Additional Information:

At the end of an article, there is a link that says "Get started with Brave Wallet" that does in fact give more information about Brave Wallet. However, I don't want to get started until I know what it is and as a result never clicked that link until this issue prompted me to address the issue like a QA tester/debugger rather than a consumer.

Miscellaneous Information:

n/a

@diracdeltas
Copy link
Member

cc @jamesmudgett

@rebron rebron added the feature/web3/wallet Integrating Ethereum+ wallet support label Nov 11, 2022
@rebron rebron added this to Web3 Nov 11, 2022
@jamesmudgett jamesmudgett self-assigned this Nov 18, 2022
@jamesmudgett jamesmudgett added the priority/P4 Planned work. We expect to get to it "soon". label Nov 18, 2022
@jamesmudgett jamesmudgett moved this to Backlog in Web3 Nov 18, 2022
@jamesmudgett
Copy link

@Douglashdaniel Douglashdaniel added QA/Yes release-notes/include front-end-change This task is a front end task and doesn't need any C++ changes labels Jan 12, 2023
@Douglashdaniel Douglashdaniel moved this from Backlog to In Progress in Web3 Jan 12, 2023
@Douglashdaniel Douglashdaniel moved this from In Progress to In Review in Web3 Jan 12, 2023
@github-project-automation github-project-automation bot moved this from In Review to Done in Web3 Jan 12, 2023
@brave-builds brave-builds added this to the 1.49.x - Nightly milestone Jan 12, 2023
@srirambv
Copy link
Contributor

Verification passed on

Brave 1.49.108 Chromium: 110.0.5481.104 (Official Build) beta (64-bit)
Revision 46de4a7f41979e829b430bc1ee30ef483aa227ac-refs/branch-heads/5481_77@{#19}
OS Linux
  • Verified steps from brave/brave-core#16644
  • Verified clicking on Learn more about Brave Wallet opens help centre link (https://support.brave.com/hc/en-us/categories/360001059151-Brave-Wallet) in a new tab
26638.mp4

Verification passed on

Brave 1.49.108 Chromium: 110.0.5481.104 (Official Build) beta (64-bit)
Revision 46de4a7f41979e829b430bc1ee30ef483aa227ac-refs/branch-heads/5481_77@{#19}
OS Windows 11 Version 22H2 (Build 22621.755)
  • Verified steps from brave/brave-core#16644
  • Verified clicking on Learn more about Brave Wallet opens help centre link (https://support.brave.com/hc/en-us/categories/360001059151-Brave-Wallet) in a new tab
26638.mp4

Verification passed on

Brave 1.49.108 Chromium: 110.0.5481.104 (Official Build) beta (arm64)
Revision 46de4a7f41979e829b430bc1ee30ef483aa227ac-refs/branch-heads/5481_77@{#19}
OS macOS Version 13.0 (Build 22A380)
  • Verified steps from brave/brave-core#16644
  • Verified clicking on Learn more about Brave Wallet opens help centre link (https://support.brave.com/hc/en-us/categories/360001059151-Brave-Wallet) in a new tab
26638.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/web3/wallet Integrating Ethereum+ wallet support front-end-change This task is a front end task and doesn't need any C++ changes OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/include
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

7 participants