mirror of
https://github.com/floccusaddon/floccus.git
synced 2026-04-25 22:26:06 +03:00
[GH-ISSUE #180] Usability issues, Firefox DE & Vivaldi & Chromium Inox on Arch, using WebDAV #170
Labels
No labels
browser-specific
bug
correctness issues
enhancement
feature: Google Drive
feature: Linkwarden
feature: git
feature: nextcloud-bookmarks
feature: tabs
feature: webdav
help wanted
native-app
priority: high
priority: low
priority: medium
pull-request
question
question
stale
upstream
waiting for more information
wontfix
🙁 Not following issue template
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/floccus#170
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @lightdot on GitHub (Oct 2, 2018).
Original GitHub issue: https://github.com/floccusaddon/floccus/issues/180
First, thank you all for your time and efforts. A project like this is sorely needed.
These are the results of my quick usability research today. Perhaps it will be useful to someone.
Software versions
My primary browser is Firefox DE, bookmarks have folders and separators, no tags.
When bookmarks are natively exported to JSON, the file size is a bit less than 1.5 MB, exported to HTML, file size is about 8 MB.
Steps to reproduce, 1st run, Firefox
Expected outcome, 1st run, Firefox
A "bookmarks.xbel" file created on the server.
Actual outcome, 1st run, Firefox
Intel i7-7500U pegged to the max for minutes until I finally killed the browser.
Looking at the server logs, WebDAV URL and Bookmark file path were joined without a separator, nothing was transferred to the server. Would have expected an error or a quicker timeout.
Steps to reproduce, 2nd run, Firefox
Expected outcome, 2nd run, Firefox
A "bookmarks.xbel" file created on the server.
Actual outcome, 2nd run, Firefox
The XBEL file was successfully and quickly created.
Upon inspection, existing bookmark separators are recorded as
[](data:)in the floccus.log file and actually created as<bookmark href="data:" id="<ID>"><title/></bookmark>in the XBEL.I expected separators not to be supported yet, but they do get picked up.
Steps to reproduce, 3rd run, Chromium Inox
Expected outcome, 3rd run, Chromium Inox
Local "Bookmarks bar" folder populated from the XBEL on the server.
Actual outcome, 3rd run, Chromium Inox
Bookmarks are populated, folders are preserved, but:
Bookmarks and folders seem to just get sorted alphabetically.
Caveats
If empty "data:" bookmarks are deleted in Chromium, separators get deleted in Firefox eventually. Could be expected after seeing how the separators are handled, but still not good.
Steps to reproduce, 4th run, Vivaldi
Expected outcome, 4th run, Vivaldi
Hoping for no changes on either side but expecting to have separators deleted in Firefox.
Actual outcome, 4th run, Vivaldi
Sort order in both Vivaldi and Firefox is preserved (at least for the short while I kept the test going), at least some separators are lost in Firefox after short a while. Expected, but again a show stopper.
Conclusion
Loss of the bookmarks sort order is a serious issue.
Separators aren't handled in a good way either.
As is, Floccus can't be used to keep larger categorized bookmark collections in sync between different browsers, not without data loss.
Temporary workaround
Don't try to sync the entire bookmark collection.
Create an empty "Bookmark Sync" folder and use it to shuttle new bookmarks back and forth across the void. Pick newly arrived bookmarks by hand and copy them where they belong.
If you attempt to sync more than two browsers through the same folder, be careful not to delete the new bookmarks before they are picked up on all locations.
@marcelklehr commented on GitHub (Oct 4, 2018):
First up: Thank you for the in-depth documentation of your findings!
The problem with the WebDAV URL should be relatively easy to fix. Thanks for the heads up!
Separators are indeed a problem. The API that firefox adopted from chrome doesn't even specify separators (because chrome doesn't have that feature, of course), so the firefox team had to improvise. A workaround would be to simply hide those entries in browsers that don't have separators or cannot handle them the way firefox does.
The larger problem, as you noticed, is ordering. As XBEL preserves order, this would be possible to implement, but Nextcloud bookmarks currently has no way to store ordering. See #113 for the relevant issue. I am aware of this problem and will fix this, it'll just take some time :)
@lightdot commented on GitHub (Oct 7, 2018):
The separator situation in Chromium/Chrome is unfortunate. Users have requested them on several occasions but apparently they won't be added.
There are how-to's online that suggest mitigating the lack of functionality by adding specially crafted bookmarks, using bookmarklets. These seem to be very popular with Chromium/Chrome users.
Bookmark JSON for the horizontal "separator":
generated XBEL:
In Vivaldi, separators are supported by the browser. Unsurprisingly, their actual format is a hack, similar to the above. From Vivaldi's Bookmark file:
generated XBEL:
So we have those two, and from Firefox, JSON:
generated XBEL:
Translating the above three hackish separators to XBEL's
<separator>element and back would IMHO be safe and cover a lot of users.@github-actions[bot] commented on GitHub (Mar 21, 2023):
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.