Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(optional) support for clearing duplicate bookmarks #7

Open
ghost opened this issue Feb 4, 2021 · 2 comments
Open

(optional) support for clearing duplicate bookmarks #7

ghost opened this issue Feb 4, 2021 · 2 comments

Comments

@ghost
Copy link

ghost commented Feb 4, 2021

I'll preface by saying that this feature absolutely should be optional -- Firefox doesn't prevent from creating duplicate bookmarks and users can for instance recreate ones in the toolbar for organization purposes. Clearing duplicates by default would be destructive. I recommend adding a clear warning, similar to the one recommending creating a bookmark backup.

Personally I have loads of redundant duplicates after updating everything to HTTPS, clearing UTM parameters and the Firefox reader mode URLs.

May fit between 'replace domain' and the regex tabs:

Screenshot from 2021-02-04 17-59-32

@MrOtherGuy
Copy link
Owner

I can see how deleting duplicates could be useful, but I would guess there are already existing extensions for specifically that purpose. But I'll consider this.

I really don't want to add it into the update process, so it should be it's own "tab" so that you can run remove duplicates separately.

It would also need it's own control UI, silar to selecting which boolmarks to update ala v1.0 checkboxes because one needs to be able to select which of the duplicates to keep.

@ghost
Copy link
Author

ghost commented Feb 5, 2021

I really don't want to add it into the update process, so it should be it's own "tab" so that you can run remove duplicates separately.

This is what I tried to suggest.

It would also need it's own control UI, silar to selecting which boolmarks to update ala v1.0 checkboxes because one needs to be able to select which of the duplicates to keep.

Agreed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant