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

sometimes can't select wallets when connecting to ledger nano #6168

Open
kjozwiak opened this issue Sep 25, 2019 · 14 comments
Open

sometimes can't select wallets when connecting to ledger nano #6168

kjozwiak opened this issue Sep 25, 2019 · 14 comments
Labels
bug feature/ethereum-remote-client needs-investigation A bug not 100% confirmed/fixed priority/P3 The next thing for us to work on. It'll ride the trains. QA/Test-Plan-Specified QA/Yes

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Sep 25, 2019

Description

Sometimes when you connect the Ledger Nano via brave://wallets, you won't be able to select any of the accounts that are associated with the ledger. When you run into this issue, you'll basically need to Forget this device, close brave://wallets and attempt a connection again.

Steps to Reproduce

  1. launch 0.69.128 CR: 77.0.3865.75 and open brave://wallet
  2. create a local wallet (at this point, it doesn't matter which route you take as you'll need a local wallet first)
  3. enter your password and click on Remind me later under Secret Backup Phrase (just makes reproducing quicker)
  4. select your profile image at the top right and select Connect Hardware Wallet
  5. select Ledger from the UI and click Connect

The above is basically the STR to start the process of getting your device connected. However, this only happens sometimes so the STR are not 100%.

Actual result:

selectAccountWallet

Expected result:

Should be able to select your wallets via the ledger once connected.

Reproduces how often:

I've only ran into this ~3-4 times but it's definitely a bug that happening here and there.

Brave version (brave://version info)

Brave 0.69.128 Chromium: 77.0.3865.75 (Official Build) (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS macOS Version 10.14.6 (Build 18G95)

Version/Channel Information:

  • Can you reproduce this issue with the current release? N/A (not released on release)
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the dev channel? Yes
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? N/A
  • Does the issue resolve itself when disabling Brave Rewards? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Miscellaneous Information:

CCing @brave/legacy_qa

@kjozwiak kjozwiak added bug needs-investigation A bug not 100% confirmed/fixed QA/Yes QA/Test-Plan-Specified feature/web3/wallet Integrating Ethereum+ wallet support labels Sep 25, 2019
@bbondy
Copy link
Member

bbondy commented Oct 2, 2019

Same as metamask?

@bbondy bbondy added the priority/P3 The next thing for us to work on. It'll ride the trains. label Oct 2, 2019
@kjozwiak
Copy link
Member Author

kjozwiak commented Oct 7, 2019

@bbondy not sure as this is pretty hard to reproduce. I can spend sometime tomorrow and see if I can reproduce this on MM. CCing @srirambv in case he's seen the above on MM as he also mentioned he reproduced the above in Brave when going through Crypto Wallets.

@srirambv
Copy link
Contributor

srirambv commented Oct 8, 2019

Reproduced on Linux

@kjozwiak
Copy link
Member Author

kjozwiak commented Oct 9, 2019

Reproduced on Linux

Using MM or CW?

@girino
Copy link

girino commented Oct 30, 2019

Same occurs here. OSX high sierra, Brave Version 0.70.121

@ethgasstation
Copy link

same problem here OSX, Brave V 1.0

@danielheyman
Copy link

Had the same problem with OSX Mojave, Brave v1.01. Forgetting the device and refreshing the page fixed the problem, but it isn't intuitive unless you find this Github thread.

@jakehemmerle
Copy link

Not perfect but was able to get this to work after a little while on my Ledger Nano X. Brave Dev Version 1.11.82

@lukemulks
Copy link

lukemulks commented Jul 19, 2020

Reproduced this on Linux Fedora

Brave | 1.12.87 Chromium: 84.0.4147.89 (Official Build) dev (64-bit)
-- | --
Revision | 19abfe7bcba9318a0b2a6bc6634a67fc834aa592-refs/branch-heads/4147@{#852}
OS | Linux

I was able to eventually unlock on my 3rd attempt to connect.
In each attempt, I would try to select the address from my Nano. If I was unable to, I'd disconnect and forget the device, then reattempt.

I do not recall doing anything special in order to get it to work on the 3rd attempt.

Noticeable issue on hover in dark mode, with rows turning completely white (with the text being white, the address becomes hidden on hover).

When selecting the address does not work, it appears that the feature is unable to register the click action, and that there's no keyboard equivalent to manually attempt to tab and select the radio button w/o the mouse.

It's a bit puzzling as the same steps were followed each time, but for some reason it just seemed to work on the 3rd try.

This bug is pretty rough and I suspect has a high likelihood of negatively impacting user confidence and feature adoption if encountered during first run/setup. Might be a good candidate for UX polish/prioritization if/when cycles free up for that work.

@srirambv
Copy link
Contributor

This is an annoying bug, especially while testing. Sometimes it takes me more than 10 attempts to select the address or needs multiple reconnects

Noticeable issue on hover in dark mode, with rows turning completely white (with the text being white, the address becomes hidden on hover).

@lukemulks that's a known issue which should be fixed with the final dark theme updates

@KevinGabbert
Copy link

happens to me on Brave even now You'd think this would be solved. after about 7 tries still can't click it

@srirambv
Copy link
Contributor

@ryanml any idea if this is fixed with MM V8 rebase?

@felipeasf
Copy link

The same here: MacOs Catalina 10.15.6 - Brave Version 1.12.114 Chromium: 84.0.4147.135 (Official Build) (64-bit)

@nestoregon
Copy link

Same, PopOS 20.04

@bbondy bbondy added feature/ethereum-remote-client and removed feature/web3/wallet Integrating Ethereum+ wallet support labels Jul 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature/ethereum-remote-client needs-investigation A bug not 100% confirmed/fixed priority/P3 The next thing for us to work on. It'll ride the trains. QA/Test-Plan-Specified QA/Yes
Projects
None yet
Development

No branches or pull requests