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

Highlight current day in datepickers and not only the selected day #522

Closed
jaapcrezee opened this issue Jan 24, 2019 · 6 comments · Fixed by #527
Closed

Highlight current day in datepickers and not only the selected day #522

jaapcrezee opened this issue Jan 24, 2019 · 6 comments · Fixed by #527

Comments

@jaapcrezee
Copy link

Is your feature request related to a problem? Please describe.
The current date pickers in kimai2 do not show the active day, only the selected day.

Describe the solution you'd like
Next to the selected date, please also highlight the current (wall clock) date.

Additional context
See screenshots kimai2 picker & google picker.
kimai2_picker

google_picker

@jaapcrezee
Copy link
Author

ps I think the google picker can be improved as well ;)

@kevinpapst
Copy link
Member

You could try the branch in PR #527 and let me know what you think

@kevinpapst
Copy link
Member

Still interested?

You could try the branch in PR #527 and let me know what you think

@jaapcrezee
Copy link
Author

Sorry rather busy at the moment, still interested! I will have to find out a way to test it on my docker instance.

@kevinpapst
Copy link
Member

If that takes too long, I will rather merge it and you can see it later.
As it is basically only a CSS rule, this should not need much testing.

@lock
Copy link

lock bot commented Apr 3, 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 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants