Closed Bug 762108 Opened 13 years ago Closed 10 years ago

Reader Mode: Firefox Sync support

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: rnewman, Unassigned)

References

Details

(Whiteboard: [sync-engine-addition])

This is a backward-compatible extension to the bookmark record format. Bookmarks in Fennec have a "display" column (defined in the class "BrowserContract.Combined"): 1.38 + public static final int DISPLAY_NORMAL = 0; 1.39 + public static final int DISPLAY_READER = 1; 1.43 + public static final String DISPLAY = "display"; and we have a new root with a new magic GUID: 1.20 + public static final String READING_LIST_FOLDER_GUID = "readinglist";
See Bug 762109 for the Android Sync side of this.
rnewman: should Firefox Sync ignore the reading list folder GUID (like Android) or are you asking for something more?
For the moment, Android doesn't sync "readinglist" or its children. Bug 762109 is the bug for support. For full support we want to extend the bookmark record format to support a "display" enumeration, and also add a new bookmark root on desktop. I don't know what the bug is for desktop Reader Mode/reading list. Mak? Lucas? There will need to be some coordination with Places here.
(In reply to Richard Newman [:rnewman] from comment #3) > For the moment, Android doesn't sync "readinglist" or its children. Bug > 762109 is the bug for support. > > For full support we want to extend the bookmark record format to support a > "display" enumeration, and also add a new bookmark root on desktop. I don't > know what the bug is for desktop Reader Mode/reading list. Mak? Lucas? > > There will need to be some coordination with Places here. FWIW, I think we should only consider enabling sync support for readinglist once we implement Reader Mode in desktop Firefox (i.e. Places bits and Reader front-end).
I want to make sure we don't lose this: is this still on ice?
(In reply to Nick Alexander :nalexander from comment #5) > I want to make sure we don't lose this: is this still on ice? I have no opinion on whether this is urgent or not, but we're certainly not scheduled to spend time on it right now!
WONTFIX in favor of Bug 1082011.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.