Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stats / i18n: abbreviated month names aren't localized #1224

Closed
codebykat opened this issue Dec 3, 2015 · 6 comments
Closed

Stats / i18n: abbreviated month names aren't localized #1224

codebykat opened this issue Dec 3, 2015 · 6 comments
Assignees
Labels
i18n Stats Everything related to our analytics product at /stats/ [Type] Bug

Comments

@codebykat
Copy link
Member

Reported in 2444419-t:

I recently noticed that the abbreviated month names in the stats screens aren't localized again. I've reported something similar in the past (https://en.forums.wordpress.com/topic/i18n-date-in-stats-screen) when the stats screen changed the last time. See my screenshot for further details:

calypso_stats_month-abbrev

FYI @gibrown since you fixed it last time :)

@codebykat codebykat added Stats Everything related to our analytics product at /stats/ i18n labels Dec 3, 2015
@timmyc
Copy link
Contributor

timmyc commented Mar 29, 2016

moment.js gives us a utility to properly display these localized short month names via monthsShort but it appears our translation files only have the en-us translation for this array, see fr.json for an example - the monthsShort array is:

"monthsShort":["Jan","Feb","Mar","Apr","May","Jun","Jul","Aug","Sep","Oct","Nov","Dec"]

Long month names _are_translated, but will not visually work in most areas ( like the month labels here ) so we can't go that route.

@rralian do you happen to know how we can update the monthsShort in our translation files?

@timmyc
Copy link
Contributor

timmyc commented Mar 29, 2016

Plot ( or shall I say glot ) thickens... it appears the base short month translations are just incorrect across the board, because short month names are also invalid in atlas stats:

mis_estadisticas_ _wordpress_com

@timmyc
Copy link
Contributor

timmyc commented Mar 29, 2016

And incorrect abbreviated months can also be seen in the tooltips on this chart, and on the "Streak" shown in the insights page.

@timmyc timmyc self-assigned this Mar 29, 2016
@rralian
Copy link
Contributor

rralian commented Mar 29, 2016

I think the back-end has changed around a fair bit since I've touched it last. The logic isn't in calypso itself, but rather the back-end used to generate those files. I believe there is some work in place to make use of moment's locale files within that back-end code to generate the final i18n files. You might have more luck pinging @akirk directly.

@akirk
Copy link
Member

akirk commented Apr 3, 2016

This is fixed now through an update of our WordPress month shortnames.
german-monthnames

@akirk akirk closed this as completed Apr 3, 2016
@timmyc
Copy link
Contributor

timmyc commented Apr 4, 2016

@akirk thanks for fixing this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n Stats Everything related to our analytics product at /stats/ [Type] Bug
Projects
None yet
Development

No branches or pull requests

5 participants