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

address bar input ignored if no default search engine set #3254

Closed
allenwb opened this issue Aug 18, 2016 · 9 comments
Closed

address bar input ignored if no default search engine set #3254

allenwb opened this issue Aug 18, 2016 · 9 comments

Comments

@allenwb
Copy link

allenwb commented Aug 18, 2016

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

Describe the issue you encountered:
URL typed into address bar is ignore (after typing return) if a default search engine is not set

Expected behavior:
Resolve URL and display page

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    macOS
  • Brave Version:
    Brave: 0.11.5
  • Steps to reproduce:
    1. using the Preferences>Search page, make sure that no search engine has a check mark in the default column. (I'm not sure how I initially got into this state because once I selected a default I couldn't get it back into the no-default set state.
      2.In a new tab, type a known good url such as brave.com into the address bar and then hit the return key
    2. nothing happens
    3. go back to the preferences?search page and select some search engine as the default
    4. repeat step ii
    5. The requested page is display
  • Screenshot if needed:
  • Any related issues:
@Sh1d0w
Copy link

Sh1d0w commented Aug 19, 2016

There was a bug with an earlier version of Brave, when you update the browser to a newer version it was forgetting your default engine, so you were left in a state with no default engine. But this was fixed in version 0.11.4 if I am not mistaken.

Can you tell me if you updated Brave to a new version, right before this happened?

@allenwb
Copy link
Author

allenwb commented Aug 19, 2016

Can you tell me if you updated Brave to a new version, right before this happened

Probably. I first noticed the behavior a few days ago and I had updated around the same time. I had been away for awhile so it is possible that more than one update happened before I noticed.

@mcalello
Copy link

It's definitely a problem, as I thought Brave was broken.

@luixxiul luixxiul added the needs-info Another team member needs information from the PR/issue opener. label Aug 19, 2016
@bbondy
Copy link
Member

bbondy commented Aug 20, 2016

@darkdh can we ask him to check something in his session file? how can users not have a default set?

@darkdh
Copy link
Member

darkdh commented Aug 20, 2016

@allenwb could you open ~/Library/Application Support/brave/session-store-1 and see what value is of the search.default-search-engine?

@allenwb
Copy link
Author

allenwb commented Aug 20, 2016

"search.default-search-engine":"DuckDuckGo"

But note this is after I found, by looking at the Preferences>Search Engines page, that none of the search engines had the default check next to them. I then set DuckDuckGo as the default and that corrected the problem.

I don't have the session-store-1 state from before that.

how can users not have a default set?

That would seem to be the question of the hour...

@darkdh
Copy link
Member

darkdh commented Aug 20, 2016

@allenwb thanks for pointing out the display problem which will be fixed in #3275. However, we cannot reproduce the urlbar problem you mentioned

@darkdh darkdh added this to the 0.11.6dev milestone Aug 20, 2016
@darkdh
Copy link
Member

darkdh commented Aug 20, 2016

QA steps:

  1. Make sure brave is closed
  2. Remove ~/Library/Application Support/brave/session-store-1
  3. Open brave and go to about:preferences#search
  4. Google should be the default search engine

@cndouglas
Copy link

Noting the original issue: #2956

@luixxiul luixxiul removed the needs-info Another team member needs information from the PR/issue opener. label Nov 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

8 participants