question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Please respect region time format on new tab

See original GitHub issue

Test plan

See https://github.com/brave/brave-core/pull/3256

Description

New Tab dashboard shows time in AM/PM, even though system settings are set to 24h time.

No similar issues in brave-browser. Similar issues in brave-laptop:

Steps to Reproduce

  1. Change system locale to some locale that defaults to 24 time, such as Brazilian Portuguese (Mac: System Preferences > Language & Region > Preferred languages > click the plus button)
  2. Open Brave with New Tab

Actual result:

screen shot 2018-09-26 at 17 51 20

Expected result:

Time should match locale time (24-hour if locale defaults to it, such as pt-BR suggested in the test plan above). Switching back to locales that default to 12-hour (such as en-US) should update accordingly.

Reproduces how often:

Always

Brave version (chrome://version info)

Brave | 0.55.6 Chromium: 70.0.3538.16 (Official Build) dev (64-bit) Revision | 16ed95b41bb05e565b11fb66ac33c660b721f778-refs/branch-heads/3538@{#306} OS | Mac OS X JavaScript | V8 7.0.276.9

Reproducible on current release:

Yes, reproducible also on browser-laptop.

Additional Information

After digging through the source code here: https://github.com/brave/brave-core/blob/master/components/brave_new_tab_ui/components/app.tsx I can see that brave-ui/old/clock is being used: https://github.com/brave/brave-ui/blob/master/src/old/clock/index.tsx Which renders AM/PM time directly.

Issue Analytics

  • State:closed
  • Created 5 years ago
  • Reactions:20
  • Comments:31 (6 by maintainers)

github_iconTop GitHub Comments

11reactions
mrfnassecommented, May 2, 2019

I managed to solve it by using “English United Kingdom” as language. It is better than having the 12 hour clock format.

If it is impossible to make it work by reading the locale, why not just add one of those “switches” in the settings page, where you can choose 12 or 24 hour manually?

And if not even that is possible, why not make 24 hour clock standard? There are only a few countries in the world where microsoft standard is 12 hour. 43dwH

3reactions
c-kirkebycommented, Nov 15, 2019

This is still an issue on Windows 10 Pro 1909 (18363.476) in Version 1.0.0 Chromium: 78.0.3904.97 (Official Build) (64-bit). At 15:18 the new tab page is still showing 3:18.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Time formatting on the new tab clock - Brave Community
I find programs not following my regional/formatting settings extremely annoying and somewhat insulting. At the very least, something as visible ...
Read more >
Why doesn't DateTime.ToShortTimeString() respect the Short ...
NET seems to select a short time format not based upon this setting but based upon the current culture: Region and Language ->...
Read more >
How to change date and time formats on Windows 10
Click on the Clock, Language, and Region link. Click on the Change date, time, or numbers formats link.
Read more >
Excel Date and Time Formatting - My Online Training Hub
Select the cell(s) containing the dates you want to format. · Press CTRL+1, or right-click > Format Cells to open the Format Cells...
Read more >
234897 - When printing, headers have American date format ...
Hello mcrompton, The date changes are not respective with the local computer's format. Please change the language settings in "chrome://settings -- ...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found