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

[Bug Report] Wrong week translations for hr locale #21037

Closed
primoz990 opened this issue Jun 1, 2021 · 2 comments · Fixed by #21040
Closed

[Bug Report] Wrong week translations for hr locale #21037

primoz990 opened this issue Jun 1, 2021 · 2 comments · Fixed by #21040
Assignees

Comments

@primoz990
Copy link

Screenshot 2021-06-01 at 09 38 59

Element UI version

2.15.2

OS/Browsers version

Mac OS / Chrome

Vue version

2.6.12

Reproduction Link

sun: 'Pon',
mon: 'Uto',
tue: 'Sri',
wed: 'Čet',
thu: 'Pet',
fri: 'Sub',
sat: 'Ned'

Steps to reproduce

With language set to hr(croatian) open a datepicker. The weekdays are offset by one day. The typo in the translations is clearly visible:

sun: 'Pon',
mon: 'Uto',
tue: 'Sri',
wed: 'Čet',
thu: 'Pet',
fri: 'Sub',
sat: 'Ned'

What is Expected?

The weekdays should be in this order(firstDayOfWeek set to 1):
Pon Uto Sri Čet Pet Sub Ned

What is actually happening?

They are like this:
Uto Sri Čet Pet Sub Ned Pon

@superchangme
Copy link

firstDayOfWeek set to 7 (of course,you don't need set ,7 is default ) will make Sunday at first .In China ,1 mean Monday :-)

@primoz990
Copy link
Author

firstDayOfWeek set to 7 (of course,you don't need set ,7 is default ) will make Sunday at first .In China ,1 mean Monday :-)

The issue has nothing to do with the firstDayOfWeek setting. The translations are in wrong order:

sun: 'Pon',
mon: 'Uto',
tue: 'Sri',
wed: 'Čet',
thu: 'Pet',
fri: 'Sub',
sat: 'Ned'

"Mon" in croatian(hr) should be "Pon", "Tue" -> "Uto",... and so on...
Every one that speaks any slavic language can confirm this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants