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

Add default spinner speeds #43

Merged
merged 1 commit into from
Sep 1, 2022
Merged

Add default spinner speeds #43

merged 1 commit into from
Sep 1, 2022

Conversation

chelnak
Copy link
Owner

@chelnak chelnak commented Sep 1, 2022

Prior to this PR spinners would use the default speed of 250 milliseconds set in the managers constructor method.

This commit introduces a new concept of animation properties that allow each spinner to have its own default speed. The speeds used come from this website.

It is still possible to globally override the speed via the managers WithFrameDuration option.

@chelnak chelnak added the enhancement New feature or request label Sep 1, 2022
@chelnak chelnak self-assigned this Sep 1, 2022
@chelnak chelnak force-pushed the default_spinner_speeds branch 3 times, most recently from ccbfb4d to 616aa0c Compare September 1, 2022 16:49
Prior to this commit spinners would use the default speed of 250
milliseconds set in the managers constructor method.

This commit introduces a new concept of animation properties that allow
each spinner to have it's own default speed. The speeds used come from
https://wiki.tcl-lang.org/page/Text+Spinner.

It is still possible to globally override the speed via the managers
WithFrameDuration option.
@chelnak chelnak merged commit d91aa46 into main Sep 1, 2022
@chelnak chelnak deleted the default_spinner_speeds branch September 1, 2022 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant