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

Update generic-yearly-gantt-chart-with-today.json #597

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DRVRogo
Copy link

@DRVRogo DRVRogo commented Oct 7, 2022

Update to generic-yearly-gantt-chart-with-today.json

Q A
Bug fix? no
New sample? no
Related issues?

What's in this Pull Request?

Updates generic-yearly-gantt-chart-with-today.json so that the Gantt chart now correctly renders when Start and End dates are outside the timeframe of the current year. That is, if the Start and/or End dates are before 1st January of current year, as well as if Start and/or End dates are after 31st December of current year.

I hope I'm doing this right (this is my first time editing on GitHub).  I've updated the JSON so that the Gantt chart now correctly renders when Start and End dates are outside the timeframe of the current year.  That is, if the Start and/or End dates are before 1st January of current year, as well as if Start and/or End dates are after 31st December of current year.
@tecchan1107 tecchan1107 self-assigned this Oct 17, 2022
@tecchan1107
Copy link
Collaborator

Thank you for the update and for sending the pull request @DRVRogo 🙏✨
One point of concern is noted in the following. If possible, I would appreciate a reply.

"left": "= floor(((Number(@now)-Number(Date('' + getYear([$Start]) + '/1/1')))/(1000*60*60*24))/365 * 100) + '%'"
"left": "= floor(((Number(@now)-Number(Date('' + getYear(@now) + '/1/1')))/(1000*60*60*24))/365 * 100) + '%'"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Modifying [$Start] to @now, for example, would cause a red line to appear representing today, even if the schedule is for next year.
How about reverting back to the previous state so that only this year's schedule shows the red line?
image

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 this pull request may close these issues.

None yet

2 participants