Time displayed on lock screen should follow format specified in shortTimeFormat = %I:%M %p Some locales (e.g. zh-TW) need to position AM/PM before the time (%p %I:%M) and this currently not possible.

Kaze, in bug 932270 comment 7 you wrote home screen, but I think you were referring to the lockscreen (moving to System for now). system.properties > # either 24-hour format (%H:%M) or 12-hour without AM/PM (%I:%M) > statusbarTimeFormat=%I:%M But I can't find any reference in the code for this string, while I found 3 references to shortTimeFormat and hard-coded AM/PM

Note that this bug becomes pretty urgent after bug 932270 lands: if a locale changes AM|PM, we're basically not displaying anything besides hh:mm

Yes, the `statusbarTimeFormat' was an old UX request to hide the meridiem in the statusbar — it’s obsolete now, thanks for catching this. For the System app I’d suggest to create a “pretty” time format from the common `shortTimeFormat'. Something like: var timeFormat = _('shortTimeFormat').replace('%p', '<span>%p</span>'); element.innerHTML = f.localeFormat(now, timeFormat); This would work just fine for statusbar.js and media_recording.js. For the lockscreen clock, you’ll have to: • remove the content of div.lockscreen-lock (index.html, line 1074); • modify the CSS accordingly in statusbar.css, something like: (untested) .lockscreen-clock { margin: -0.8rem 0 -1rem -0.4rem; font-weight: 300; font-size: calc(26 * 0.226rem); } .lockscreen-clock span { /* optional meridiem */ font-size: calc(17 * 0.226rem); } I hope this helps. :-)

Thanks kaze, this approach is definitely better than the one I first used. Waiting for Travis to run online on the PR, can't run it on Nightly for some reasons.

Strangely enough, travis passed. https://travis-ci.org/mozilla-b2g/gaia/builds/19653279 Note on the screenshot above: I set shortTimeFormat as "%p %I:%M" and AX/PX as localization for AM/PM

Setting 1.4? Without this change, if a locale changes the translation of AM or PM, we're not displaying the time correctly.

Comment on attachment 8382235 [details] Pull request on Github LGTM. Thanks for taking the time to polish this patch and taking reference screenshots!

Master: db6668d3ce79879b7213967ec6ef8d4a446e29e9

Verified on Keon with build 20140307, setting phone to zh-TW.