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

feat: moment marker should move dynamically over time #114

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

Conversation

L-ZJoker
Copy link

No description provided.

Copy link
Owner

@zunnzunn zunnzunn left a comment

Choose a reason for hiding this comment

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

useNow() is suitable for this use-case, however, you might want to configure this behavior to be more specific. A few things to consider:

  1. Try playing around with this. If you set a narrow time window (chart-start and chart-end) in the gantt chart, e.g. a few minutes around the current time), you will notice that the time marker does not move in a, say, 1-second-interval as you'd expect it to.
  2. What if the current time is not within the set time window and therefore not visible anyways? The periodic recomputation of the xDist of the time marker is a waste of resources then. It'd make sense to check whether the current time is within set boundaries.

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.

2 participants