Closed Bug 525184 Opened 12 years ago Closed 12 years ago
Create a virtual places root for Mobile bookmarks
Mobile bookmarks should live in a virtual root: * name is "Mobile" (localized) * direct child of the main places root, not a child of one of the other roots * has an annotation so it can be located quickly, regardless of l10n Fennec will use this root as the parent for all bookmarks created in Fennec. Default bookmarks will also live in the root.
This could impact Weave too
Looks like it's as simple as this. JSON importer already allows creating bookmarks whose parent are the places root, so we just need to tweak the default bookmarks file accordingly. Will also need to talk to the weave folks about what this means for them. Put the "mobileRoot" property on the BookmarkList object for easy access - kind of random, but couldn't think of a better place to put it. Can't run the tests at the moment (Mac), so I just made obvious changes by inspection. I noticed some of them are obsoleted by e.g. bug 516958 so I guess they aren't currently passing?
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Attachment #411147 - Flags: review?(mark.finkle)
Comment on attachment 411147 [details] [diff] [review] patch Can we use mobile/bookmarksRoot instead of fennec/bookmarksRoot for the annotation r+ with that Maybe we could add mobileRoot to PlaceUtils? The Weave team might find that useful too.
Attachment #411147 - Flags: review?(mark.finkle) → review+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → Post-B5
You need to log in before you can comment on or make changes to this bug.