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

Bookmarking page without title #8912

Closed
NejcZdovc opened this issue May 17, 2017 · 5 comments
Closed

Bookmarking page without title #8912

NejcZdovc opened this issue May 17, 2017 · 5 comments
Assignees

Comments

@NejcZdovc
Copy link
Contributor

NejcZdovc commented May 17, 2017

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

  • Describe the issue you encountered:
    When you bookmark page that don't have a title, bookmark name is empty

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    all

  • Brave Version (revision SHA):
    ced8add

  • Steps to reproduce:

    1. go to page without title (test/fixtures/page_no_title.html)
    2. click on star icon (bookmark)
  • Actual result:
    Bookmark name is empty

  • Expected result:
    Bookmark name should be url without http/s. If tab is a local file we should display file name.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    yes

  • Is this an issue in the currently released version?
    yes

  • Can this issue be consistently reproduced?
    yes

#2972

@NejcZdovc NejcZdovc added this to the 0.15.400 milestone May 17, 2017
@NejcZdovc NejcZdovc self-assigned this May 17, 2017
@kumarrishav kumarrishav self-assigned this May 17, 2017
@NejcZdovc
Copy link
Contributor Author

NejcZdovc commented May 17, 2017

@kumarrishav I am already working on this one, sorry 😃 Will add you as a reviewer to my PR, ok?

@luixxiul
Copy link
Contributor

Something might have regressed after #3442 was fixed.

@kumarrishav
Copy link
Contributor

@NejcZdovc sure. Actually, i assigned myself to track this issue. There are so many issues that selected issues get lost generally. Assigning is a good way to track though
Thanks

@bsclifton
Copy link
Member

bsclifton commented May 17, 2017

@kumarrishav you might consider using notifications (whether you use them in GitHub or via email). I find those super useful to track issues. I'm subscripted to the whole repo though and I have various email filters

@NejcZdovc
Copy link
Contributor Author

I notice that this is related to this bug #8908, so will be closing this one for now.

@NejcZdovc NejcZdovc removed this from the 0.15.400 milestone May 18, 2017
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