Closed
Bug 1201109
Opened 10 years ago
Closed 10 years ago
Intermediate step: expose server bookmark mirrors as read-only bookmarks in the UI
Categories
(Firefox for iOS :: Data Storage, defect)
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| fxios | 1.1+ | --- |
People
(Reporter: rnewman, Assigned: rnewman)
References
(Blocks 1 open bug)
Details
This involves implementing BookmarkModelFactory on top of the mirror storage introduced in Bug 1201104.
There will be some UX work involved too. The first thing I expect is that we'll need a new state for the bookmark star: "bookmarked on desktop". This is a star you can't toggle yet.
This bug gets us 80% of the value of syncing bookmarks with 20% of the effort, so it's worth the detour in case Bug 1141850 doesn't get done in time.
|
Assignee | |
Comment 1•10 years ago
|
||
My short-term hack for the bookmark star is to simply ignore desktop bookmarks altogether. This matches our revised Aha card of "it's just a tree in the home panel".
I have this mostly done.
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Component: Sync → Data Storage
|
|
Assignee | |
Comment 2•10 years ago
|
||
f5ad0aa
Flipped around the dependency; Emily can finish up Bug 1205924 on top of this work, and we'll live with butt-ugly desktop bookmarks for now.
|
|
Assignee | |
Comment 3•10 years ago
|
||
I manually verified that desktop bookmarks aren't deletable, but mobile bookmarks are. This seems to work fine without my editable changes that stick around in f0d3044aebf7ce150ae175f36e41ef7da7130d36, but we can cherry-pick that if necessary.
You need to log in
before you can comment on or make changes to this bug.
Description
•