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

Wallet "Visible assets" dialog: set the focus in the search field #19790

Closed
tredondo opened this issue Nov 28, 2021 · 2 comments
Closed

Wallet "Visible assets" dialog: set the focus in the search field #19790

tredondo opened this issue Nov 28, 2021 · 2 comments
Labels
closed/duplicate Issue has already been reported feature/web3/wallet Integrating Ethereum+ wallet support good first issue OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes

Comments

@tredondo
Copy link

Description

When the user clicks the "+ Visible assets" button in the Portfolio, it would help to set the focus in the search field of the popup dialog.

Steps to Reproduce

  1. Go to brave://wallet/crypto/portfolio
  2. Click "+ Visible assets"

Actual result:

Expected result:

image

Brave version (brave://version info)

Brave 1.32.113 Chromium: 96.0.4664.45 (Official Build) (64-bit)
Revision 76e4c1bb2ab4671b8beba3444e61c0f17584b2fc-refs/branch-heads/4664@{#947}
OS Linux

Version/Channel Information:

Version 1.32.113 Chromium: 96.0.4664.45 (Official Build) (64-bit)

@srirambv srirambv added feature/web3/wallet Integrating Ethereum+ wallet support good first issue QA/Yes labels Nov 29, 2021
@Douglashdaniel
Copy link

Currently have an issue pending for this! 🙂 #19326

@bbondy
Copy link
Member

bbondy commented Nov 30, 2021

Closing this one as a duplicate since it'll be covered in the above issue that Douglas mentioned. Thanks for posting though!

@bbondy bbondy closed this as completed Nov 30, 2021
@bbondy bbondy added closed/duplicate Issue has already been reported priority/P3 The next thing for us to work on. It'll ride the trains. labels Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/duplicate Issue has already been reported feature/web3/wallet Integrating Ethereum+ wallet support good first issue OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes
Projects
None yet
Development

No branches or pull requests

4 participants