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

Calendar date picker not using i18n setting #1572

Closed
noyez opened this issue Mar 19, 2020 · 3 comments · Fixed by #1952
Closed

Calendar date picker not using i18n setting #1572

noyez opened this issue Mar 19, 2020 · 3 comments · Fixed by #1952
Milestone

Comments

@noyez
Copy link

noyez commented Mar 19, 2020

Describe the bug
When user preference is set to English, so first day of week is Sunday, the date-picker still uses Monday as first day of week, although the main calendar view displays the correct order of days. See screenshot.

To Reproduce

  1. Set users's preference to Enlish so calendar starts on Sunday
  2. Add entry in timesheet.
  3. Click on From field to bring up calendar
  4. Observer calendar has first day of week as Monday.

Screenshots

Screen Shot 2020-03-19 at 11 31 12

Desktop/Smartphone
Please complete the following information if this seems to be a frontend issue:

  • Firefox 74.0

Additional context
Add any other context about the problem here.

  • Kimai version: 1.8
  • PHP version: php-7.2
@kevinpapst
Copy link
Member

English is just the translation, not the locale. Not all English speaking countries use Sunday as first day of the week. I'd say it's quite the opposite as many users have to use English, as Kimai is not translated to their own language.

Probably the first day of the week should be configurable, or in a more global scale => the users country / locale ...

But you are right that is should be consistent.
I am not sure why that is different, maybe the datepicker doesn't respect the first day of week setting from moment.js.

@kevinpapst
Copy link
Member

I believe this was fixed in 1.9.
But with the next release I added a user specific setting for the "first day of week", see #1952 - so I am linking this issue with PR.

@kevinpapst kevinpapst added this to the 1.11 milestone Sep 5, 2020
@github-actions
Copy link

github-actions bot commented Oct 7, 2021

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. If you use Kimai on a daily basis, please consider donating to support further development of Kimai.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging a pull request may close this issue.

2 participants