Skip to content
This repository has been archived by the owner. It is now read-only.

Bookmarks in Other Bookmarks should not be merged into Bookmark Toolbar #7231

Closed
luixxiul opened this issue Feb 14, 2017 · 2 comments
Closed

Bookmarks in Other Bookmarks should not be merged into Bookmark Toolbar #7231

luixxiul opened this issue Feb 14, 2017 · 2 comments

Comments

@luixxiul
Copy link
Contributor

@luixxiul luixxiul commented Feb 14, 2017

Describe the issue you encountered: By default the bookmarks in Other Bookmarks are merged into Bookmark Toolbar when you import browser data. It surely could work well in some cases (say you would find bookmarks which you think you have lost), but other cases not so well. If you for example have a number of the bookmarks in Other Bookmarks subfolder, they also will be imported on the bookmark toolbar.

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

  • Brave Version (revision SHA): 0.13.2

  • Steps to reproduce:

    1. Bookmark URLs on Bookmark Toolbar
    2. Move some of them into Other Bookmarks on Bookmark Manager
    3. Export them
    4. Import them, merging into Bookmark toolbar
  • Actual result: All bookmarks are imported on the toolbar

  • Expected result: Only the bookmarks which were bookmarked on the toolbar should be imported to the toolbar. The others should be imported into Other Bookmarks

  • Extra QA steps:
    1.
    2.
    3.

  • Any related issues:

@darkdh
Copy link
Member

@darkdh darkdh commented Feb 14, 2017

This is the same behavior as Chrome except Chrome doesn't merge them into bookmarks toolbar.
But we will do the same thing in #7194

@luixxiul
Copy link
Contributor Author

@luixxiul luixxiul commented Jun 4, 2017

Closing in favor of #7194

@luixxiul luixxiul closed this Jun 4, 2017
@luixxiul luixxiul removed the suggestion label Jun 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.