Closed
Bug 669909
Opened 13 years ago
Closed 13 years ago
Add "..." to menu items under Manage Account
Categories
(Firefox :: Sync, defect)
Tracking
()
VERIFIED
FIXED
mozilla8
People
(Reporter: djst, Assigned: philikon)
Details
(Whiteboard: [verified in services])
Attachments
(2 files)
5.54 KB,
image/png
|
Details | |
2.92 KB,
patch
|
mconnor
:
review-
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•13 years ago
|
||
Good catch, David. Easy enough to fix.
Updated•13 years ago
|
Attachment #544513 -
Flags: review?(rnewman) → review+
Assignee | ||
Comment 2•13 years ago
|
||
http://hg.mozilla.org/services/services-central/rev/da8c5f9c9c44
Whiteboard: [fixed in services]
Comment 3•13 years ago
|
||
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-
Assignee | ||
Comment 4•13 years ago
|
||
Uh yeah you're right, Mike. Pushed follow-up as discussed on IRC: http://hg.mozilla.org/services/services-central/rev/d3dd1a9d6dce
Comment 5•13 years ago
|
||
yay </petpeeve>
Comment 6•13 years ago
|
||
verified with s-c build of 20110711
Whiteboard: [fixed in services] → [verified in services]
Assignee | ||
Comment 7•13 years ago
|
||
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
Updated•13 years ago
|
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
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.
Description
•