-
Notifications
You must be signed in to change notification settings - Fork 24.5k
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
Include the NativeAnimation module on iOS in the starter project #10638
![:octocat: :octocat:](https://github.githubassets.com/images/icons/emoji/octocat.png)
Comments
![:octocat: :octocat:](https://github.githubassets.com/images/icons/emoji/octocat.png)
I'm new to the codebase and would like to take this on if that's alright. I looked around and couldn't find where this logic lives. Perhaps if you have a minute, can you point out where/how it is included on Android? Also, some steps to test expected behavior. Should there be an automated test? Thanks!! |
@janicduplessis I've made a pull request here. I added the Do let me know if everything is OK. I published my branch in Sinopia and tried having |
Fixes 'useNativeDriver' warning. Refs: facebook/react-native#10638
Summary: Fixes facebook#10638. Added NativeAnimation library to the starter project iOS generator. Passing `useNativeDriver: true` to `Animated` config will enable the app to tap into the native code for animations. **Test plan** Init a RN project and animate an element. Enable native driver as follows: ``` Animated.timing( this.state.value, { toValue: 300, // some value useNativeDriver: true } ).start(); ``` Earlier, this used to crash. Closes facebook#10783 Differential Revision: D4159386 Pulled By: mkonicek fbshipit-source-id: 993481a31b4446eab24ef4ee35ae1941d7a7aae9
![:octocat: :octocat:](https://github.githubassets.com/images/icons/emoji/octocat.png)
https://github.com/facebook/react-native/tree/master/Libraries/NativeAnimation should be included when creating a new project. We will want to use native animations in react-native core so having it included by default makes sense. It is also included by default on Android now.
The text was updated successfully, but these errors were encountered: