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

Long-term fixes to lottie-scroller #10

Open
4 tasks
MinamiFunakoshiTR opened this issue Oct 7, 2021 · 0 comments
Open
4 tasks

Long-term fixes to lottie-scroller #10

MinamiFunakoshiTR opened this issue Oct 7, 2021 · 0 comments
Assignees

Comments

@MinamiFunakoshiTR
Copy link
Contributor

MinamiFunakoshiTR commented Oct 7, 2021

  • It's not great that this is using window scroll events instead of IntersectionObserver. Ultimately, use svelte-scroller to handle the scroll detection. The progress binding is what you'd use to calculate the seconds that drive the video.
  • Vanilla rewrites of things like a simple debounce function -- this should just rely on lodash-es
  • I'm curious about the small animation. Does this basically mean you get two breakpoints? Should we plan for future where you might need more? Not sure.
  • Small performance improvement, but I noticed this uses the path config option to pass the lottie data to bodymovin. BUT if we used animationData and directly passed in the JSON data to this component we could save a step waiting for bodymovin to load it and set all the heights for the steps immediately. That'll also make it easy to implement with svelte-scroller if we go that route.
  • [] I'm curious how many types of layout this will support. Maybe in addition to widthRatio there are cases you'd want to set an explicit height?? Not sure...
  • [] Make container width adjustable with wide/wider/widest/fluid etc
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

No branches or pull requests

2 participants