Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Address bar should get focus when launching Brave #5079

Closed
bsclifton opened this issue Oct 24, 2016 · 4 comments
Closed

Address bar should get focus when launching Brave #5079

bsclifton opened this issue Oct 24, 2016 · 4 comments
Labels
accessibility polish Nice to have — usually related to front-end/visual tasks. suggestion

Comments

@bsclifton
Copy link
Member

Did you search for similar issues before submitting this one?
Yes

Describe the issue you encountered:
Launching Brave does NOT set focus to the address bar unless it opens to about:newtab

Expected behavior:
No matter what page is loaded by default, the address bar should have focus

@bsclifton bsclifton added polish Nice to have — usually related to front-end/visual tasks. accessibility labels Oct 24, 2016
@Sh1d0w
Copy link

Sh1d0w commented Oct 24, 2016

Hm, this is not the behaviour of Chrome. I think that the UrlBar should gain focus only when Brave starts with empty new tab, to promt you to eneter address into it.

Otherwise imagine following scenario:

  1. I have set Brave's home page to my fav site.
  2. I launch Brave and when my home page loads instead of using down arrow to scroll my fav website I have first to use the mouse to click out of the UrlBar because the webview does not have focus because of the url bar.

Same applies if you set Brave's home page to my fav search engine.

Instead of start typing directly into the loaded search engine's input in the page (like it is now), I will be forced to type in the url bar initially. Which kills the idea of having search engine as home page, no? :)

@bsclifton
Copy link
Member Author

@Sh1d0w that is a great point- I didn't bother to try Chrome for comparison

cc: @bradleyrichter what do you think?

@bradleyrichter
Copy link
Contributor

Yes. Sorry I was not more explicit.

This is only for a new tab. It will happen on first time use and if the user has prefs set to open without recalling tabs from last session.

On Oct 24, 2016, at 12:19 AM, Brian Clifton notifications@github.com wrote:

@Sh1d0w that is a great point- I didn't bother to try Chrome for comparison

cc: @bradleyrichter what do you think?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@bsclifton
Copy link
Member Author

Ah ok- this should be working then 😄 I'll close this out, but please reopen if you can reproduce with 0.12.6

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accessibility polish Nice to have — usually related to front-end/visual tasks. suggestion
Projects
None yet
Development

No branches or pull requests

3 participants