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

Improve Bookmark Import - request from community #9569

Closed
LaurenWags opened this issue Jun 19, 2017 · 6 comments
Closed

Improve Bookmark Import - request from community #9569

LaurenWags opened this issue Jun 19, 2017 · 6 comments

Comments

@LaurenWags
Copy link

@LaurenWags LaurenWags commented Jun 19, 2017

Please create a better way to import bookmarks. User requests that they not all be in an 'Imported' folder, but rather the imported bookmarks have the same structure as the browser that they came from.

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

  • Brave Version (revision SHA):

  • Steps to reproduce:

    1. Import bookmarks.
    2. Bookmarks are found under an 'Imported' folder, instead of having the same structure that they do from their source.
  • Actual result:
    Bookmarks are found under an 'Imported' folder.

  • Expected result:
    Bookmarks would not be under an 'Imported' folder, they would have the same structure as their source.

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

  • Is this an issue in the currently released version?

  • Can this issue be consistently reproduced?

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:

  • Any related issues:

@benmordecai
Copy link

@benmordecai benmordecai commented Aug 13, 2017

Most people importing bookmarks to Brave are attempting to migrate browsers, so the intuitive behavior would be for the imported results to mirror the appearance of the browser of origin.

@luixxiul
Copy link
Contributor

@luixxiul luixxiul commented Aug 14, 2017

iirc that was introduced intentionally, cc @darkdh

@darkdh
Copy link
Member

@darkdh darkdh commented Aug 14, 2017

The issue which remove the function is here #7194
firefox and chrome both put imported bookmarks into a separate folder which will be easier for users to organize their bookmarks by their own way
@bsclifton , @bradleyrichter

@luixxiul
Copy link
Contributor

@luixxiul luixxiul commented Aug 14, 2017

wontfix?

@luixxiul luixxiul added the needs-info label Aug 14, 2017
@bsclifton
Copy link
Member

@bsclifton bsclifton commented Aug 15, 2017

++ on wontfix for me 👍

@darkdh darkdh added the wontfix label Aug 15, 2017
@darkdh
Copy link
Member

@darkdh darkdh commented Aug 15, 2017

I prefer wonfix

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
6 participants
You can’t perform that action at this time.