If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

simplify bookmarks hierarchy

VERIFIED FIXED in fennec1.0b3

Status

Fennec Graveyard
Bookmarks
VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: madhava, Assigned: vingtetun)

Tracking

(Blocks: 1 bug)

Trunk
fennec1.0b3
x86
Mac OS X
Dependency tree / graph
Bug Flags:
wanted-fennec1.0 ?

Details

Attachments

(2 attachments)

(Reporter)

Description

8 years ago
By default, there should be no hierarchy to the list of bookmarks in Fennec.  So, for example, there should be no node (and therefore titlebar) called Unsorted Bookmarks shown that currently contains all bookmarks (see attached image).  From an implementation perspective, this may just mean not displaying that node titlebar.

When synced with weave, there will be hierarchy introduced because it exists in desktop Firefox.  Ideally, what a user would see post-sync is, first, a list of their bookmarks created on the mobile device, and then a folder called something along the lines of "synced bookmarks" which could contain the hierarchy inherited from the desktop (unsorted, bookmarks menu, and bookmarks toolbar).
Flags: wanted-fennec1.0?
(Reporter)

Updated

8 years ago
tracking-fennec: --- → ?

Updated

8 years ago
tracking-fennec: ? → 1.0+

Updated

8 years ago
Assignee: nobody → gavin.sharp
Blocks: 515394
I split the "Synced Bookmarks" part of this bug to bug 515394 - It's too much
work for one bug.
Created attachment 399494 [details]
Hierarchy

Madhava, Is it ok for you?
Attachment #399494 - Flags: ui-review?(madhava)
Created attachment 399695 [details] [diff] [review]
Patch v0.1
Attachment #399695 - Flags: review?(mark.finkle)
Assignee: gavin.sharp → 21
Comment on attachment 399695 [details] [diff] [review]
Patch v0.1

We should change arrowtop-16 to arrowup-16, but I can do that when landing.
Attachment #399695 - Flags: review?(mark.finkle) → review+
(Reporter)

Updated

8 years ago
Attachment #399494 - Flags: ui-review?(madhava) → ui-review+
pushed with tweaks:
https://hg.mozilla.org/mobile-browser/rev/e4165af4a150
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
veriifed FIXED on build:

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.2a2pre) Gecko/20090914
Fennec/1.0b4pre
Status: RESOLVED → VERIFIED
Component: General → Bookmarks
You need to log in before you can comment on or make changes to this bug.