Commit 2406f15c authored by Jimmy Gong's avatar Jimmy Gong Committed by Commit Bot

Update locale time format in print jobs

- Previously print jobs that are ongoing or printed within 24 hours
  displayed the printed time in a 12hr format.
- Not all language locales are in the 12hr format, so the fix is to
  remove that parameter and use the numeric locale format.

Before: https://screenshot.googleplex.com/B3Y9JiAp6ET
After: https://screenshot.googleplex.com/srMiTVBJT6G

Bug: 1106231
Change-Id: I4103e954b5b3b26b6eacc9b188366ee022e5e10b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2311616Reviewed-by: default avatarKyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Jimmy Gong <jimmyxgong@chromium.org>
Cr-Commit-Position: refs/heads/master@{#791015}
parent 14dc401c
......@@ -318,7 +318,7 @@ Polymer({
// |jsDate|'s date, display the 12hour time of the current date.
if (isToday(jsDate)) {
return jsDate.toLocaleTimeString(/*locales=*/undefined,
{hour12: true, hour: 'numeric', minute: 'numeric'});
{hour: 'numeric', minute: 'numeric'});
}
// Remove the day of the week from the date.
return jsDate.toLocaleDateString(/*locales=*/undefined,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment