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

Bookmarked website behaviour in autocomplete. #5024

Closed
Sh1d0w opened this issue Oct 21, 2016 · 1 comment · Fixed by #5084
Closed

Bookmarked website behaviour in autocomplete. #5024

Sh1d0w opened this issue Oct 21, 2016 · 1 comment · Fixed by #5084

Comments

@Sh1d0w
Copy link

Sh1d0w commented Oct 21, 2016

Did you search for similar issues before submitting this one?

No.

Describe the issue you encountered:
If you often visit a website and you bookmark it it won't appear anymore in the url bar suggestions in the History section, if you have turned off "Show bookmark suggestions" in settings.

Expected behavior:

Now how I understand those settings:

Since you can turn on/off each section in the url bar autocomplete - Tab matches, Bookmar matches and History matches (since it takes a lot of space if you enable them on) by turning off the "Show bookmark matches" setting, I just want the bookmarks section in url bar suggestions to be hidden. But it still have to show all matches from the History, regardless if the history entry is bookmarked or not.

Obviosly Brave don't think so and hides all history matches in the history section if the entry is bookmarked, which is very frustrating, because I am forced to enable the setting in order to see the website in the autocomplete and the "highlight" text feature.

The correct behaviour should be as follows: Do not exclude any entries from history matches if the "Show bookmarked matches" setting is turned off. If it is turned on, the logic should be as it is now. That would solve the problem.

  • Platform (Win7, 8, 10? macOS? Linux distro?): All
  • Brave Version: 0.12.6-Preview 2
  • Steps to reproduce:
    1. Visit github.com several times and confirm it appears under History section in url bar suggestions
    2. Bookmark it and confirm it properly appears under bookmarks section in url bar suggestions
    3. Now from settings disable "Show bookmarks suggestions" for the autocomplete
    4. Type github.com in the url bar and observe it does not appear under history section as it should
  • Screenshot if needed:
  • Any related issues:
@aekeus
Copy link
Member

aekeus commented Oct 24, 2016

Thanks for the detailed bug report. I have a fix in progress for this issue.

cc @bbondy

aekeus added a commit that referenced this issue Oct 24, 2016
Bookmark sites are by definition also history items. Removing them from the
history section of the auto-complete causes them not to be selected when
a matching auto-complete term is entered.

This commit fixes that behaviour.

Auditors: @bbondy

Test Plan: (Courtesy of @Sh1d0w)

  * Visit github.com several times and confirm it appears under History section in url bar suggestions
  * Bookmark it and confirm it properly appears under bookmarks section in url bar suggestions
  * Now from settings disable "Show bookmarks suggestions" for the autocomplete
  * Type github.com in the url bar and ensure it appears under history section as it should

Fixes: #5024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants