Skip to content

Conversation

logustra
Copy link
Contributor

Motivation

#1678

Test plan

Nothing visual has changed

@logustra logustra changed the title fix: button animation Fix: button animation Mar 31, 2020
@callstack-bot
Copy link

callstack-bot commented Mar 31, 2020

Hey @logustra, thank you for your pull request 🤗. The documentation from this branch can be viewed here.

@Trancever
Copy link
Contributor

@logustra Is it possible to animate elevation using native driver now?

Copy link
Contributor

@jayu jayu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! @Trancever I just checked and it seems to work with expo 34
https://snack.expo.io/@jaysbytes/react-native-animated-example

@vicary
Copy link

vicary commented Apr 4, 2020

From #959 (comment), we should be cautious to put a simple true. Something like what ad516cc does should be more sensible, because the referenced issue only wants to get rid of a warning where undefined should be explicitly false.

@logustra
Copy link
Contributor Author

logustra commented Apr 5, 2020

@Trancever
I have checked it with expo 32 and got an error on iOS and I fixed it like you did on ad516cc

@Trancever Trancever changed the base branch from master to 4.0 April 8, 2020 11:11
@Trancever
Copy link
Contributor

@logustra I believe it's safe to pass true for iOS cause we are dropping support for RN < 0.60 anyway.

@Trancever Trancever merged commit 4b400d6 into callstack:4.0 Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants