Closed Bug 669909 Opened 13 years ago Closed 13 years ago

Add "..." to menu items under Manage Account

Categories

(Firefox :: Sync, defect)

All
Windows 7
defect
Not set
minor

Tracking

()

VERIFIED FIXED
mozilla8

People

(Reporter: djst, Assigned: philikon)

Details

(Whiteboard: [verified in services])

Attachments

(2 files)

Attached image Menu items lack "..."
The drop-down menu for the button Manage Accounts under the Options window contains four items that all show you a separate dialog. As such, the menu item captions should end with "..."

This is especially important for an item like "Reset Sync" as the current menu caption makes it look like the account will be reset immediately.
Attached patch v1Splinter Review
Good catch, David. Easy enough to fix.
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #544513 - Flags: review?(rnewman)
Attachment #544513 - Flags: review?(rnewman) → review+
Comment on attachment 544513 [details] [diff] [review]
v1

Bzzt, incorrect ellipsis usage detected.

"Opens a new window" is the wrong heuristic.  "Is an action requiring further info or confirmation" is closer to the truth, though GNOME draws that line a little differently.

Relevant links:

http://msdn.microsoft.com/en-us/library/aa511502.aspx#ellipses
http://developer.apple.com/library/mac/#documentation/UserExperience/Conceptual/AppleHIGuidelines/XHIGText/XHIGText.html#//apple_ref/doc/uid/TP30000365-TPXREF126
http://developer.gnome.org/hig-book/stable/menus-design.html.en#menu-item-types

View Quota and My Sync Key shouldn't be changed to add an ellipsis.
Attachment #544513 - Flags: review+ → review-
Uh yeah you're right, Mike. Pushed follow-up as discussed on IRC:

http://hg.mozilla.org/services/services-central/rev/d3dd1a9d6dce
yay

</petpeeve>
verified with s-c build of 20110711
Whiteboard: [fixed in services] → [verified in services]
http://hg.mozilla.org/mozilla-central/rev/da8c5f9c9c44
http://hg.mozilla.org/mozilla-central/rev/d3dd1a9d6dce
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Status: RESOLVED → VERIFIED
Component: Firefox Sync: UI → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: