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

Critical paths lines stay partially gray #1433

Closed
fabiojpoli opened this issue Aug 31, 2020 · 2 comments
Closed

Critical paths lines stay partially gray #1433

fabiojpoli opened this issue Aug 31, 2020 · 2 comments
Assignees
@fabiojpoli
Labels
bug Something isn't working resolved Fixed but not yet released (available in the nightly builds)
Milestone
4.0.0

Comments

@fabiojpoli
Copy link

Steps to reproduce:

  1. open advanced demo
  2. click "Critical paths" button
  3. drag "Install Apache" task to start on Feb 18, 2019
  4. the task is highlighted reddish as critical but it is partially gray

91470368-9efc5f00-e862-11ea-8b75-efdd5d951e9a
@fabiojpoli fabiojpoli added the bug Something isn't working label Aug 31, 2020
@fabiojpoli fabiojpoli self-assigned this Aug 31, 2020
@fabiojpoli fabiojpoli mentioned this issue Aug 31, 2020
Closed
@fabiojpoli fabiojpoli added in progress ready for review Issue is fixed, the pull request is being reviewed and removed in progress labels Aug 31, 2020
@fabiojpoli
Copy link
Author

Fixed but found another moment that the path change to gray when drag "Contact Designer" to left

@fabiojpoli fabiojpoli added in progress and removed ready for review Issue is fixed, the pull request is being reviewed labels Sep 10, 2020
@isglass isglass added this to the Gantt 4.0.0 milestone Sep 11, 2020
@fabiojpoli fabiojpoli mentioned this issue Sep 16, 2020
Closed
@fabiojpoli
Copy link
Author

based on my tests (sort, collapse, expand, move), it works fine for advanced demo (http://lh/master/bryntum-suite/Gantt/examples/advanced/)
but for php demo it doesn’t (http://lh/master/bryntum-suite/Gantt/examples/php/)
So I have created a new issue specifically about this demo then we can merge the current PR
#1524

@fabiojpoli fabiojpoli added ready for review Issue is fixed, the pull request is being reviewed resolved Fixed but not yet released (available in the nightly builds) and removed code review ready for review Issue is fixed, the pull request is being reviewed labels Sep 16, 2020
@SergeyMaltsev SergeyMaltsev modified the milestones: Gantt 4.0.0, 4.0.0 Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fabiojpoli fabiojpoli

Labels
bug Something isn't working resolved Fixed but not yet released (available in the nightly builds)
Projects
None yet
Milestone
4.0.0
Development

No branches or pull requests
3 participants
@fabiojpoli @isglass @SergeyMaltsev