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

Feature/tooltip autoscroll #468

Merged
merged 7 commits into from
Jul 30, 2024
Merged

Feature/tooltip autoscroll #468

merged 7 commits into from
Jul 30, 2024

Conversation

tonihele
Copy link
Owner

@tonihele tonihele commented Jul 28, 2024

Little bit quirky. Implements the tooltip autoscroll if it doesn't entirely fit to the allocated space.

  • Somehow I could not get the style applied properly, so the component now has a fixed style set. Not a problem for us but annoying design
  • The effect length is 70s, hmm, yeah, the effects time calculation is a bit difficult for this kind of task. I would have wanted infinite time but then we just get 1s always as normalized time. So now everything is normalized to this 70s, the delays and speed. Pretty annoying. The effect length should perhaps be take account in the math so that this length variable could be adjusted freely without the effect speed changing....

Edit: I fixed the math so that there are far less variables and more constants

@tonihele tonihele merged commit e2ecdd5 into master Jul 30, 2024
3 checks passed
@tonihele tonihele deleted the feature/tooltip-autoscroll branch July 30, 2024 17:53
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.

1 participant