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.

Sign up for GitHub

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

Jump to bottom

Make Gantt chart date inclusive #868

Closed
FranklinYu opened this issue Jul 1, 2019 · 2 comments
Closed

Make Gantt chart date inclusive #868

FranklinYu opened this issue Jul 1, 2019 · 2 comments
Assignees
@ThePenguin1140
Labels
Graph: Gantt Status: Approved Is ready to be worked on Type: Enhancement New feature or request

Comments

@FranklinYu
Copy link

This is a feature request.

It would be cool if we can configure our Gantt chart so that the dates are inclusive. For example if there is a task from June 1st to June 3rd I can write 2019-06-01, 2019-06-03. Currently the ending date is exclusive, so I have to write 2019-06-01, 2019-06-04 which is counter-intuitive.

It can be either a switch in JavaScript API, or some command in the diagram definition (similar to dateFormat and title).
@ThePenguin1140 ThePenguin1140 self-assigned this Jul 6, 2019
@ThePenguin1140 ThePenguin1140 added the Status: Approved Is ready to be worked on label Jul 6, 2019
@ThePenguin1140 ThePenguin1140 mentioned this issue Jul 7, 2019
Merged
4 tasks
@IOrlandoni
Copy link
Member

Hey, @ThePenguin1140. This is done and we can close it, right?

@FranklinYu
Copy link
Author

Verified with ./dist/index.html. Looking forward to next release!

@dependabot dependabot bot mentioned this issue Mar 16, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ThePenguin1140 ThePenguin1140

Labels
Graph: Gantt Status: Approved Is ready to be worked on Type: Enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@IOrlandoni @FranklinYu @ThePenguin1140