Closed Bug 845096 Opened 12 years ago Closed 11 years ago

Make sure sync UI dialogs have proper strings / format

Categories

(Firefox for Metro Graveyard :: Theme, defect)

All
Windows 8.1
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jimm, Assigned: mbrubeck)

Details

(Whiteboard: [metro-mvp?])

Attachments

(1 file)

> 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.)
Is there anything left to do here? Seems like we got what we wanted in bug 841511.
Flags: needinfo?(jmathies)
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)
<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">
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!
Attached patch patch (deleted) — Splinter Review
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Attachment #724977 - Flags: review?(jmathies)
Attachment #724977 - Flags: review?(jmathies) → review+
https://hg.mozilla.org/mozilla-central/rev/4d6bd84ed4fd
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
No longer blocks: metrov1triage
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: