Closed
Bug 1098468
Opened 10 years ago
Closed 9 years ago
»Show all Bookmarks« should show unsorted bookmarks by default
Categories
(Firefox :: Bookmarks & History, defect, P1)
Tracking
()
VERIFIED
FIXED
Firefox 46
Tracking | Status | |
---|---|---|
firefox46 | --- | verified |
People
(Reporter: phlsa, Assigned: dao)
References
Details
(Whiteboard: [Onboarding])
Attachments
(2 files)
90.06 KB,
image/png
|
Details | |
1.98 KB,
patch
|
mak
:
review+
|
Details | Diff | Splinter Review |
Currently, whenever users open the bookmarks manager, they are presented with only the top level items of the bookmarks hierarchy (which never directly includes bookmarks). To make this slightly more useful, we should make the bookmarks manager show »Unsorted Bookmarks« by default. Reasons: - When inexperienced users search for their bookmarks (which go to unsorted by default), they often click Show all Bookmarks (Michael Verdi has done a user study that shows this behavior) - More experienced users have filed a way a bookmark at some point and will probably spot the folders in the sidebar (which is a common place for folders)
Flags: firefox-backlog+
Comment 1•10 years ago
|
||
Looking at the user research, looks like users always fallback to Show all bookmarks when they can't find the starred bookmark in the menu, that issue seems to have an higher priority. Fwiw, what is requested here should be easy to do from a code point of view, could take more time just to fix tests.
Points: --- → 3
Flags: qe-verify+
Updated•9 years ago
|
Priority: -- → P3
Comment 3•9 years ago
|
||
People are often confused when they open the bookmarks menu, select "Show All Bookmarks" and then don't see their bookmarks in the Library window. To add to the confusion, people often click on the folders listed on the right side of the window which doesn't do them any good. We should change the default selection to be unsorted bookmarks so that the bookmarks that people have created and not filed anywhere in particular are shown when opening the Bookmarks Library window.
Updated•9 years ago
|
Priority: P3 → P2
Updated•9 years ago
|
Whiteboard: [Onboarding]
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → dao
Assignee | ||
Comment 4•9 years ago
|
||
So this would make "Show All Bookmarks" and "Unsorted Bookmarks" do the same thing. Should we keep both menu items?
Flags: needinfo?(philipp)
Flags: needinfo?(mverdi)
Comment 5•9 years ago
|
||
Unsorted Bookmarks is a submenu in the bookmarks menu button. it's just we never synced up the main menu bar structure.
Assignee | ||
Comment 6•9 years ago
|
||
I was referring to the "Unsorted Bookmarks" menu item in the Bookmarks menu.
Assignee | ||
Comment 7•9 years ago
|
||
That said, the "Unsorted Bookmarks" menu from the menu button is also interesting. There's certainly some extra redundancy there as well if "Show All Bookmarks" goes to unsorted bookmarks.
Reporter | ||
Comment 8•9 years ago
|
||
In the bookmarks button menu, the two items are definitely different enough (one opening a menu), so let's not touch unsorted bookmarks there. In the menu bar, I'm a bit surprised that unsorted bookmarks doesn't open a submenu as well. My initial reaction would be to make the item in the menu bar behave the same way as the item in the button menu. But let's sort out that situation in a separate bug and not block this work on it.
Flags: needinfo?(philipp)
Assignee | ||
Comment 9•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=7dcfd24614ee
Flags: needinfo?(mverdi)
Attachment #8707893 -
Flags: review?(mak77)
Comment 10•9 years ago
|
||
Comment on attachment 8707893 [details] [diff] [review] patch Review of attachment 8707893 [details] [diff] [review]: ----------------------------------------------------------------- Unfortunately "All Bookmarks" will still have the wrong meaning since this view has no toolbar/menu... but let's see how it feels. In the end we are not requiring more clicks than before to reach those other roots and that's positive.
Attachment #8707893 -
Flags: review?(mak77) → review+
Updated•9 years ago
|
Priority: P2 → P1
Comment 12•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/be4d89e61e5b
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox46:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
Comment 13•9 years ago
|
||
I was able to reproduce this issue on Firefox 45.0a2 (2016-01-18) and on Windows 10 x86. The issue was verified on Firefox 46.0a2 (2016-01-27) across platforms[1] and is no longer reproducible. [1] - Windows 10x86, Ubuntu 14.04 x86 and Mac OS X 10.9.5
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•