Move "Sync Now" to the left side of the Firefox menu

RESOLVED FIXED

Status

()

Firefox
Menus
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: faaborg, Assigned: Margaret)

Tracking

unspecified
x86
Windows 7
Points:
---
Bug Flags:
in-litmus ?

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [target-betaN])

Attachments

(1 attachment, 2 obsolete attachments)

We initially placed the "Setup Sync" and "Sync Now" commands on the right side of the Firefox menu, at the bottom.

The idea was partly to draw more attention to this command by isolating it, and also to group it with the items on the right side that all represent the types of data that are actually being sync'd (bookmarks, history, preferences, etc.)

However, it just ends up looking kind of strange and out of place over there.  Let's move it to the left side and place it after Full Screen and before Exit (it's important that it remains close to Exit since a lot of users are going to want to sync before the close).
On a somewhat related note, we might want to consider putting "Tabs From Other Computers" in a somewhat more accessible location on Windows than just the "All Tabs" drop down. Would that be a candidate for the Firefox menu as well?
(Reporter)

Comment 2

7 years ago
With 29% of users accessing the tab overflow menu, I think that's going to be discoverable enough (placing it in the Firefox only doubles the chance of discovery to 59%).  Also it's nice to group all of the tab features together into a single place.
(Reporter)

Comment 3

7 years ago
actually I was filtered on beginner data, either way the value are about the same when you look at all levels: https://heatmap.mozillalabs.com
(Assignee)

Comment 4

7 years ago
Created attachment 496532 [details] [diff] [review]
patch
Assignee: nobody → margaret.leibovic
Status: NEW → ASSIGNED
Attachment #496532 - Flags: review?(dao)
We should move Full Screen up to a different group. Grouping it with Sync and Exit doesn't seem to make sense.

Updated

7 years ago
Blocks: 556174
Component: Firefox Sync: UI → Menus
Product: Mozilla Services → Firefox
QA Contact: sync-ui → menus
Flags: in-litmus?
(Reporter)

Comment 6

7 years ago
>We should move Full Screen up to a different group. Grouping it with Sync and
>Exit doesn't seem to make sense.

Yep I agree.  Note that if we add the zoom controls those would end up being grouped with full screen:

Web Developer
--------------
Zoom [slider control]
Full Screen
--------------
Sync Now
Exit
(Assignee)

Comment 7

7 years ago
Created attachment 496666 [details] [diff] [review]
patch v2

Added menuseparator.
Attachment #496532 - Attachment is obsolete: true
Attachment #496666 - Flags: review?(dao)
Attachment #496532 - Flags: review?(dao)
Summary: Move "Sync Now" to the left side of the FIrefox menu → Move "Sync Now" to the left side of the Firefox menu
Whiteboard: [target-betaN]
Since we don't have the zoom control yet, should we move Full Screen somewhere else for the time being? Without the zoom item, it seems that we'd get too many separators for too few items again.
Comment on attachment 496666 [details] [diff] [review]
patch v2

See previous comment. Please don't assume the zoom item to exist when implementing this.
Attachment #496666 - Flags: review?(dao)
(Assignee)

Comment 10

7 years ago
Alex, what do you think I should do until the zoom control exists? No separator? That would look like this:

Web Developer
--------------
Full Screen
Sync Now
Exit

Or, we could move Full Screen somewhere else like Dão suggested in comment 8.
This bug is being tracked as part of this set: http://areweprettyyet.com/4/syncPromotion/
(Assignee)

Comment 12

7 years ago
Created attachment 503517 [details] [diff] [review]
patch v3

I got rid of the menuseparator to match Alex's mockup.
Attachment #496666 - Attachment is obsolete: true
Attachment #503517 - Flags: review?(dao)
Comment on attachment 503517 [details] [diff] [review]
patch v3

Well, I can't complain about the code. FWIW, the grouping doesn't seem to make sense.
Attachment #503517 - Flags: review?(dao) → review+
Attachment #503517 - Flags: approval2.0+
Since we don't have the zoom control, we either have to have too many separators (comment #8), or groups that don't really make that much sense.  I mostly indifferent, with perhaps a slight preference to limiting the number of separators for the time being and having the last group not make that much sense.
(Assignee)

Comment 15

7 years ago
http://hg.mozilla.org/mozilla-central/rev/c53984e562b8

I added a comment to bug 592147 about adding the menuseparator once the zoom controls are added.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.