-
Notifications
You must be signed in to change notification settings - Fork 183
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
Possibility to not rely on js thread? #99
Comments
I m in holidays but will take a look soon concerning the native computation or at least defering the animation only when it s possible on the JS thread. Thanks for your feedback 😊 |
I didn't forget about this issue, I'm looking for time to take it 😅 |
Any movement on this? |
I'm not working on this for now, but if anybody wants to offload the animation to the native thread, feel free to open a PR 🙏 |
react-native-reanimated v2 does exactly this. |
Yep, thanks for the feedback 😊 . I'm not very interested in adding a new dependency here since the placeholder animations are basic one and that we can avoid relying on the JS thread using react native "Animated" module. However, I didn't try to create a custom animation with react-native-reanimated v2, that might be interesting 😊 |
Would a PR where all the animations with |
Yes this is an approach that I would prefer over adding react-native-reanimated :). Nice idea @stoica94 ! 😊 |
Great! I will provide a PR this weekend and we will discuss further then 👍 |
Thank you so much for this 🚀 |
To give some feedbacks on that: #134 (comment) |
Would this still get merged? |
It won't until it works on Android :/ |
Reanimated 2 does exactly this. I'd suggest a branch for reanimated 2 would be great,ps if you haven't tried reanimated 2 you have no idea what you're losing it's just a lot easier a lot performant, and you write a lot of less code for the same thing, also the syntax is identical to what you normally write for your js calculations and it uses a few hooks to handle stuff.JUST AMAZING! |
I'm the maintainer of a new library called Moti. It's like framer motion for React Native (+Web). It's powered by Reanimated 2. I also have a skeleton component, which should solve the issue at hand. Docs: moti.fyi/skeleton |
It's a shame this didn't get fixed the moti library is also absolutely amazing and could easily be able to re create these animations but it means a few more dependencies |
Feel free to fork or to switch if this does not fit your needs 😊. Kind reminder that I m not paid to provide you this library for free. |
Sure, it was just a kind suggestion performance wise, but you don't have to do anything for anyone. I'll make my own no need to fork, thanks anyways for sharing this library. |
Sorry for being harsh. "It s a shame" does not translate very well in my native language 😅 Thanks for the suggestion however. I will probably try to request help from maintainers that will, hopefully, be more available and interested in maintaining this project since I dont have the time nor the energy to 🙂. Sorry for the previous message, I misinterpreted what you said. |
No hard feeling, what it's a shame meant was this is a really good library with a lot of stars and users but unfortunately it has a little performance problem which is not resolved yet. Sorry I should've been more clear. Thanks again for this after all, this got me the idea on how to build my placeholder component's animations. |
First, thank you for this awesome library, really useful for us.
However, recently I am having a problem to display the placeholder animation smoothly since the js thread is occupied performing other computation. The animation even stopped when such condition happens. Is there a way to display smooth animation regardless how busy the js thread is?
The whole idea of loading animation or placeholder is to show users some animation when we are preparing the data or layout, when the animation stutter this gives bad impression to users.
Thanks in advance
The text was updated successfully, but these errors were encountered: