Closed
Bug 845096
Opened 12 years ago
Closed 12 years ago
Make sure sync UI dialogs have proper strings / format
Categories
(Firefox for Metro Graveyard :: Theme, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jimm, Assigned: mbrubeck)
Details
(Whiteboard: [metro-mvp?])
Attachments
(1 file)
(deleted),
patch
|
jimm
:
review+
|
Details | Diff | Splinter Review |
Reporter | ||
Comment 1•12 years ago
|
||
> All of these should be covered by flyout bug 841511:
>
> http://mxr.mozilla.org/mozilla-central/source/browser/metro/locales/en-US/
> chrome/sync.dtd
(except the sentences, which I won't be editing.)
Updated•12 years ago
|
Blocks: metrov1triage
Comment 2•12 years ago
|
||
Is there anything left to do here? Seems like we got what we wanted in bug 841511.
Updated•12 years ago
|
Flags: needinfo?(jmathies)
Reporter | ||
Comment 3•12 years ago
|
||
The only thing I see is a button that might need a case change. Probably should walk through it all and make sure everything looks right to be sure.
sync.setup.tryagain=Try again
Flags: needinfo?(jmathies)
Reporter | ||
Comment 4•12 years ago
|
||
<yuan> jimm: we use sentence case for hyperlinks and button labels, title case for titles. Hyperlink is the proper control for "Learn more". "Learn more" directs the users to another place and dismiss the current UI.
So I guess sync.setup.tryagain is correct, but there are couple others that are wrong.
sync.update.button=Learn More
<!ENTITY sync.syncNow "Sync Now">
Comment 5•12 years ago
|
||
Sorry if I was not clear about the capitalization previously.
We are following the MS guide:http://msdn.microsoft.com/en-us/library/windows/desktop/aa511453.aspx
"Use sentence-style capitalization for labels. Doing so is more appropriate for Windows tone and the use of short phrases for command buttons."
So, I believe we need to change to "Pair a device", "Sync now", "Set up Sync". Thanks!
Assignee | ||
Comment 6•12 years ago
|
||
Reporter | ||
Updated•12 years ago
|
Attachment #724977 -
Flags: review?(jmathies) → review+
Assignee | ||
Comment 7•12 years ago
|
||
Hardware: x86_64 → All
Comment 8•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
No longer blocks: metrov1triage
Updated•10 years ago
|
OS: Windows 8 Metro → Windows 8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•