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

Default bookmarks show in history portion of autocomplete #5892

Closed
bridiver opened this issue Nov 29, 2016 · 5 comments
Closed

Default bookmarks show in history portion of autocomplete #5892

bridiver opened this issue Nov 29, 2016 · 5 comments

Comments

@bridiver
Copy link
Collaborator

bridiver commented Nov 29, 2016

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

Describe the issue you encountered:
Installing Brave fresh will create a new session file which will be pre-populated with 6 Brave Software related accounts. These have lastAccessedTime set and therefore are considered history entries

autocomplete shows https://www.facebook.com/BraveSoftware/ when typing facebook.com. This is quite annoying because I want to go to the main facebook page at https://www.facebook.com/ and autocomplete forces me to delete BraveSoftware from the url to do that

Expected behavior:
only urls I actually enter should become part of autocomplete

cc @cezaraugusto

@cezaraugusto cezaraugusto self-assigned this Nov 29, 2016
@pixelass
Copy link

I had similar issues when I activated "tab-completion". I deactivated it since it totally messed with the address bar.
This is especially annoying when you have pinned tabs. I was basically unable to go to the github homepage because it kept redirecting me to my profile which is the pinned tab.

@bridiver
Copy link
Collaborator Author

@pixelass the pinned tab thing is a slightly different issue. This issue is specifically targeted at urls that show up in autocomplete because they are "top sites" in the default new tab page. I believe there is a separate issue for autocomplete "overcompleting" by default which I also agree is annoying. Safari does the same thing and I think Chrome as well, but my preference would be to autocomplete the shortest match

@pixelass
Copy link

pixelass commented Nov 30, 2016

@bridiver thanks for explaining. I read this issue and thought "this sounds very familiar" ;). Thought it would help.

So I guess you are talking about the following issue (occurs with bookmarks but probably behaves as you describe)

The gif shows me trying to go to codepen.io but it keeps completing and even changing cursor position when I simply use the arrow keys.

complete

@bsclifton
Copy link
Member

bsclifton commented Dec 1, 2016

(updated the title / initial post with more info)

@bridiver would the discussion in #5767 be enough to resolve this issue? Basically picking the shortest match, as you suggest above?

If so, would we be able to close this issue? I think it's OK to have the entries in history, as long as autocomplete prefers the root domain over the bookmarked URL

@bsclifton bsclifton changed the title newtab urls show up in autocomplete Default bookmarks show in history portion of autocomplete Dec 1, 2016
@bsclifton
Copy link
Member

I'll close this for now, since elegant handling of the URL bar would fix this issue, IMO (see notes above). @bridiver please feel free to re-open

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

4 participants