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

Autocomplete - wrong website title #4792

Closed
Sh1d0w opened this issue Oct 14, 2016 · 7 comments
Closed

Autocomplete - wrong website title #4792

Sh1d0w opened this issue Oct 14, 2016 · 7 comments

Comments

@Sh1d0w
Copy link

Sh1d0w commented Oct 14, 2016

Did you search for similar issues before submitting this one?
No.

Describe the issue you encountered:

Autocomplete yields wrong title for some history entries.

Expected behavior:

The title of the page should match the path of the page.

  • Platform (Win7, 8, 10? macOS? Linux distro?): all
  • Brave Version: 0.12.5-1
  • Steps to reproduce:
    1. Open new blank tab.
    2. Type twitter.com hit enter
    3. Open second tab start typing twitter verify that the title is correct.
    4. Now close the second tab and in the first opened tab with twitter, click on a feed post
    5. Notice that the url and the title changes
      6 .Again open a new tab and start typing twitter
    6. Observe that the title is wrong
  • Screenshot if needed:
  • Any related issues:
    Hidden history entries appear by deleting the latest entry #4224
    URL bar sometimes shows a part of domain #4663
@bbondy
Copy link
Member

bbondy commented Oct 15, 2016

I assume this is a sites entry that has the wrong title, can you check in session-store-1 in your brave user profile dir?
If that's it, do you have STR to get one oft hose site entries?

@Sh1d0w
Copy link
Author

Sh1d0w commented Oct 16, 2016

@bbondy This is very easy reproducible.

  1. Open new blan tab.
  2. Type twitter.com hit enter
  3. Open second tab start typing twitter verify that the title is correct.
  4. Now close the second tab and in the first opened tab with twitter, click on a feed post so a popup appears and you can preview the comments
  5. Notice that the url and the title changes
  6. Again open a new tab and start typing twitter
  7. Observe that the title for the main domain twitter.com is wrong

I even know why it is happening. It is because of this line:

https://github.com/brave/browser-laptop/blob/master/js/components/frame.js#L698

Here you update the page title, but in this case I think the history entry should not be updated unless you monitor the URL change as well? I think Brave is not handling historyPopState correctly.

@cndouglas cndouglas changed the title Autocomplete - wrong wesbite title Autocomplete - wrong website title Oct 16, 2016
@dhaneshsabane
Copy link

Wasn't able to reproduce this bug on Fedora.

@bbondy bbondy added this to the 0.12.7dev milestone Oct 18, 2016
@bbondy bbondy modified the milestones: 0.12.8dev, 0.12.7dev, 0.12.9dev Oct 24, 2016
@bsclifton
Copy link
Member

Removing milestone for now. Let's chat if you'd like to see this be bumped up in priority. Thanks 😄

@bsclifton bsclifton removed this from the 0.12.9dev milestone Nov 8, 2016
@garrettr
Copy link

I can reproduce this on Mac OS 10.10.5 with Brave 0.12.15. I noticed it when auto-completing entries with fuzzy matching in the address bar. The fuzzy-match query would correctly match against the URL, but the correct title for the matched URL would not be displayed.

Here's an example screenshot—notice that the titles don't match the URLs for the first two history entries:

screen shot 2016-12-30 at 1 12 26 pm

@luixxiul luixxiul added this to the Backlog milestone Jan 29, 2017
@bsclifton
Copy link
Member

This appears to have been fixed- I can't reproduce with the steps mentioned

@bsclifton bsclifton removed this from the Backlog milestone May 17, 2017
@luixxiul
Copy link
Contributor

I cannot find a proper STR but I experience the issue on Windows. I notice it on youtube especially, wondering if it could be related with youtube autoplay.

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

6 participants