Gamote / lottie-react

A lightweight React library for rendering complex After Effects animations in real time using Lottie.
https://lottiereact.com
Other
772 stars 59 forks source link

animationData shouldn't be required #16

Open n0uki opened 3 years ago

n0uki commented 3 years ago

Hi, the animationData property shouldn't be mandatory since we can only use the path property. Lottie documentation states that only one of two should be required. https://github.com/airbnb/lottie-web/wiki/loadAnimation-options#path-or-animationdatamandatory-at-least-one

Path is easier to use since it doesn't require to import the json file. Please consider to change that in future release :)

Gamote commented 3 years ago

@n0uki Noted, this will be added in the v3.

liho00 commented 2 years ago

@Gamote Why is this being removed?

Gamote commented 2 years ago

Hi @liho00 this is not being removed. It was added in the v3 branch. If you want it in the v2 you are more than welcome to contribute.

hotoo commented 9 months ago

@Gamote lottie-react@v2 dependencies on lottie-web@^5.10.2, lottie-web is support path, why lottie-react@v2 cannot support it?