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

Fix: Adds support for Visua11y's 'no animations' option (fixes #3) #19

Merged
merged 17 commits into from
Sep 26, 2023

Conversation

swashbuck
Copy link
Contributor

@swashbuck swashbuck commented Sep 7, 2023

Fixes #3

Fix

  • Adds support for Visua11y's "no animations" option. Requires framework v5.31.27 due to the DOM Element Modification API dependency
  • Fixes issue when using "_playFirstViewOnly": true and scrolling off and back again. The frame displayed when back on screen was blank. May not be an issue (see notes).

Notes

For some reason, I needed to subtract one from the total frames in order to go to the last frame.

this.animation.totalFrames - 1

This was causing an issue with _playFirstViewOnly which has been fixed. It would be good to know if the problem was with the specific .svgz file that I was using, or if others can reproduce the same issue.

In any case, would it be safer to just stop at the 2nd to last frame in case any given .svgz file has the same issue? I don't know how often this issue comes up.

@swashbuck swashbuck added bug Something isn't working enhancement New feature or request labels Sep 7, 2023
@swashbuck swashbuck self-assigned this Sep 7, 2023
@swashbuck swashbuck changed the title Fix: (fixes #3) Fix: Adds support for Visua11y's 'no animations' option (fixes #3) Sep 7, 2023
@swashbuck swashbuck changed the title Fix: Adds support for Visua11y's 'no animations' option (fixes #3) Breaking: Adds support for Visua11y's 'no animations' option (fixes #3) Sep 7, 2023
js/LottieView.js Outdated Show resolved Hide resolved
js/LottieView.js Outdated Show resolved Hide resolved
js/LottieView.js Outdated Show resolved Hide resolved
@swashbuck swashbuck changed the title Breaking: Adds support for Visua11y's 'no animations' option (fixes #3) Fix: Adds support for Visua11y's 'no animations' option (fixes #3) Sep 21, 2023
@oliverfoster
Copy link
Member

Beautiful. So easy to read. Thanks.

@oliverfoster oliverfoster merged commit 7d60ab0 into master Sep 26, 2023
@github-actions
Copy link

🎉 This PR is included in version 2.0.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Visua11y no animation support
2 participants