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

Edit bookmark doesn't show bookmark folder #8840

Closed
ayumi opened this issue May 11, 2017 · 6 comments
Closed

Edit bookmark doesn't show bookmark folder #8840

ayumi opened this issue May 11, 2017 · 6 comments
Labels
bug duplicate Issue has already been reported feature/bookmarks

Comments

@ayumi
Copy link
Contributor

ayumi commented May 11, 2017

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

  • Describe the issue you encountered:
    When editing a bookmark which is in a folder, the editor doesn't show the folder.

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

  • Brave Version (revision SHA):
    master @ 12199f9

  • Steps to reproduce:

    1. Go to twitter.com
    2. Bookmark it
    3. Create a bookmark folder
    4. Edit twitter.com bookmark to be in the bookmark folder and hit Done to save
    5. Again edit twitter.com bookmark
    6. Observe the bookmark's folder
  • Actual result:
    Folder shows as "Bookmarks Toolbar"

  • Expected result:
    Show the bookmark folder

  • 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

@ayumi ayumi changed the title Edit bookmark doesn't show the folder for bookmarks in folders Edit bookmark doesn't show bookmark folder May 11, 2017
@kumarrishav
Copy link
Contributor

kumarrishav commented May 12, 2017

Unable to reproduce on
Brave 0.15.2
rev af7ef42

I am able to get expected folder name. Can you give more input on this? @ayumi

@NejcZdovc
Copy link
Contributor

NejcZdovc commented May 12, 2017

@kumarrishav this could be a regress in 0.15.30x, I would suggest that you checkout master and test it out there.

@luixxiul
Copy link
Contributor

luixxiul commented May 29, 2017

reproduced on 0.15.310

assuming this shares the root cause with #8022

@luixxiul
Copy link
Contributor

@srirambv is this the same issue you captured on #7032?

@srirambv
Copy link
Collaborator

@luixxiul That is correct.. Its the same issue as #7032

@srirambv
Copy link
Collaborator

Closing as dupe of #7032.

@srirambv srirambv added the duplicate Issue has already been reported label May 29, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug duplicate Issue has already been reported feature/bookmarks
Projects
None yet
Development

No branches or pull requests

5 participants