search

mifi / react-lottie-player

Fully declarative React Lottie player
MIT License
495 stars 52 forks source link

Add accurate type definitions #32

Closed kcirtaptrick closed 3 years ago

kcirtaptrick commented 3 years ago

Added type definitions, some of which mirror lottie-web Also resolves microbundle issue with this comment

Resolves issue: #5, Previous unmerged PRs: #9, #23

jamiehaywood commented 3 years ago

@mifi - having reviewed all 3 PRs, this is the best PR to merge. I think you should merge this as a starting point and then accept PRs to this file moving forward. cc: @skoruba @gugahoi

mifi commented 3 years ago

thanks @kcirtaptrick and @jamiehaywood