-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Use date display field for card dates #14619
Use date display field for card dates #14619
Conversation
bcf3468
to
ac2aa3c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The timezone issue is correctly fixed. However, there are some things that do not work correctly/like before:
- When start and end date are the same, the card abbeviated that before. Now it writes the date twice:
- When there is only a start date set, the card should show DATE - . Now it is only showing the dash:
6e16e37
to
8af42d2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
d558019
to
e548171
Compare
@HDinger I've restored previous behavior with the exception of when the dates are equal. It is reduced to a single value then which I think makes sense. |
I am not so sure about this, especially since the highlighting is lost. Before the, due date was highlighted if it is today or in the past. With the current solution this is lost. I guess it is sufficient if we re-add the highlighting to the date, but I guess we should double check with the product team. |
e6683f8
to
0ffbc5a
Compare
https://community.openproject.org/work_packages/51858