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

Expose speedMultiplier property #423

Closed

Conversation

tguerin
Copy link
Contributor

@tguerin tguerin commented Sep 29, 2024

Currently, updating the playback speed requires creating a custom render object, which unfortunately restricts the use of existing RiveAnimation utilities, such as RiveAnimation.asset.

By exposing the speedMultiplier property, we provide a straightforward way to control playback speed without affecting the existing API, as its default value is 1.0. This enhancement will greatly benefit users who wish to customize animation playback to suit their specific needs.

@HayesGordon
Copy link
Contributor

Hi @tguerin, thanks for the contribution.

We use a mono repository, so to add this change, I will need to co-author it with you. Your original commit will be preserved.

@HayesGordon
Copy link
Contributor

Closing this PR. @tguerin your commit should be visible downstream soon. We've also release v0.13.17 with this included.

rive-engineering pushed a commit that referenced this pull request Nov 15, 2024
Community contribution: #423

This also came from an internal request.

Diffs=
38c0d394bc feat: expose speed multiplier for flutter runtime (#8571)
527276dee4 Dart bump to 3.5.0 (#8550)
cda96cff1b Nnnn joystick not advancing (#8567)
6f1304ef0b Add a skeleton workflow for android gms on browserstack (#8566)
b9a4114eba Render gms to the screen when the test app is launched without a harness (#8559)
51c3bbcde2 return keep going when a state machine has changed its state (#8564)
f34822a124 accept constraints without targets (#8552)

Co-authored-by: Gordon <[email protected]>
Co-authored-by: tguerin <[email protected]>
@tguerin
Copy link
Contributor Author

tguerin commented Nov 15, 2024

Awesome! Thanks!

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