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

New time picker not working on languages without updated translation #546

Closed
infeeeee opened this issue Feb 8, 2019 · 5 comments
Closed

Comments

@infeeeee
Copy link
Contributor

infeeeee commented Feb 8, 2019

Describe the bug
The new time picker not working with languages which didn't translated the new time picker yet. It works with english or german languages.

To Reproduce
Steps to reproduce the behavior:

  1. Change the language to something which is not english or german
  2. Go to My times or Export.
  3. It shows the entries from the current month correctly as the default timeframe, but if you click on anything in the time picker nothing changes. Other filters are working as expected.

Expected behavior
It should work with translations which are not updated yet, with fallback to english.

Desktop (please complete the following information):

  • OS: Win10
  • Browser: chrome, firefox

Smartphone (please complete the following information):
I didn't check it on phone, but it should exist there.

Additional context
This bug is reproducible on the demo installation as well.

P.S. Sorry for not finding this bug when I tested this before merge.

@kevinpapst
Copy link
Member

Thanks, I'll have a look.
That "videomod.js" is definitely not coming from Kimai, do you probably have Video Player plugins activated in Firefox?

@kevinpapst kevinpapst added the bug label Feb 8, 2019
@infeeeee
Copy link
Contributor Author

infeeeee commented Feb 8, 2019

Yes it was from an addon, I deleted those parts from the bug report.

@kevinpapst
Copy link
Member

kevinpapst commented Feb 8, 2019

Seems it fails only for some languages:

  • fr
  • ar
  • hu

These worked for me (most of them are missing translations as well, so it must be something else):

  • en
  • de
  • it
  • es
  • ru
  • pt_BR

@kevinpapst
Copy link
Member

Arabic was a bit harder to fix then french and hungarian. Its done, wanna test it?

@lock
Copy link

lock bot commented Apr 9, 2019

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.

@lock lock bot locked and limited conversation to collaborators Apr 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

2 participants