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

Stuck while drawing Gantt chart with a large time span #3274

Open
purocean opened this issue Aug 1, 2022 · 3 comments
Open

Stuck while drawing Gantt chart with a large time span #3274

purocean opened this issue Aug 1, 2022 · 3 comments
Labels
Graph: Gantt roadmap items to add to roadmap for auto workflow Type: Bug / Error Something isn't working or is incorrect

Comments

@purocean
Copy link

purocean commented Aug 1, 2022

To Reproduce
Steps to reproduce the behavior:
Rendering a Gantt chart with a large time span.

Expected behavior
Limit the maximum time span to avoid getting stuck.

Screenshots

Code Sample

https://mermaid.live/edit#pako:eNptzUEKgzAQheGrhFlH0EChZGdpT9DtbAYz2oCZlDgWinj3Coqr7h4_H7wFuhwYPAwkqigTdxqzmBalNb4rUS1t5cPWmMBTY42rnavqa1U39pwXlNsf7XZ94IACFhKXRDFsjwuKMQj64sQIfpuBe5pHRUBZNzq_Ayk_QtRcwPc0TmyBZs3Pr3Rn2NU90lAoHXX9AbkhRCI

gantt
section A
A :crit,active,  des1, 2022-08-01,2022-08-05
B :crit,active,  des2, 202208-01,2d

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context

while (d.valueOf() <= maxTime) {

@purocean purocean added Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect labels Aug 1, 2022
@knsv knsv removed the Status: Triage Needs to be verified, categorized, etc label Aug 15, 2022
@tommoor
Copy link
Contributor

tommoor commented Dec 20, 2022

This feels like a potential security issue of sorts, completely locks up the browser tab

@fuzyll
Copy link

fuzyll commented Sep 19, 2023

Just ran into this over the weekend and would like to give it a bump. It's a massive usability issue when what you're trying to do surrounds a year boundary. Any time you have a "2023" you need to change to a "2024", if you backspace the "3", you run the risk of the entire tab crashing.

@nirname
Copy link
Contributor

nirname commented Oct 5, 2023

There was a test that was created while we were trying to address this issue. Unfortunately, changes didn't work as expected, so that the issue is still relevant, the test fails:

gantt
title A long Gantt Diagram
dateFormat   YYYY-MM-DD
axisFormat   %m-%d
tickInterval 1day
excludes     weekends

section Section
A task           : a1, 9999-10-01, 30d
Another task     : after a1, 20d
section Another
Task in sec      : 2022-10-20, 12d
another task     : 24d

Because the problem is still present we will comment out this test, and later one any contributor can give it a shot and try fixing it.

@nirname nirname mentioned this issue Oct 5, 2023
4 tasks
@jgreywolf jgreywolf added the roadmap items to add to roadmap for auto workflow label Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Graph: Gantt roadmap items to add to roadmap for auto workflow Type: Bug / Error Something isn't working or is incorrect
Projects
None yet
Development

No branches or pull requests

7 participants