-
-
Notifications
You must be signed in to change notification settings - Fork 867
Disable overshootClamping for small bottom sheets to avoid Reanimated issue #2387
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
base: master
Are you sure you want to change the base?
Conversation
With this fix it partially works, but the starting animation is gone meaning when you open the sheet you arent getting the smooth animation (basically same as animateOnMount=false) |
actually my whole fix is not needed, the issue is in the new react-native-reanimated withSpring, try to play with the animation configuration and you will see that it works |
@sharifhh oh hmm, can you share what steps i have to do get this working? |
these are the default configs
now
are removed from withSpring so ignore them, and just set overshootClamping as false and it will be resolved. I found the issue in reanimated and starting a thread there, will keep you posted and link it here |
Thanks for the effort. Is there any quickfix patch that can be applied while we wait, I would need this to be fixed ASAP. EDIT: Ah just set overshootClamping: false and it seems to work fine. Thanks. |
Perfect, thanks mate. fix #2272 |
Thanks, setting overshootClamping: false in animation config fixed this for me as well. Before my sheets would blink quickly and not appear or just not appear at all. |
Thanks, I apply this changes locally with patch and now modals start working on iOS after migration to new versions |
Thanks you!!! |
OMG thanks |
Hi!
Reanimated v4 introduced a bug in withSpring
software-mansion/react-native-reanimated#7947
software-mansion/react-native-reanimated#7803
this is a temp solution to make this library work for small bottom sheets