-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Modal React Native is break when using useAnimatedStyle in component #6659
Comments
This one is reported in react-native-screens and in Expo as well. I've done some research and created a reproduction here. A report was made in the RNScreens issue that this was working when using After investigating the commits between the two releases I was able to find the offending PR here: This one is containing an optimization to how shadow nodes are updated and if we revert this optimization (even tried reverting it from 3.16.1 by doing some manual copy/paste) the issues is no longer happening. bartlomiejbloniarz - would you be able to take a look? This is related to the new architecture only. |
@chrfalch Thanks for this investigation! I will take it from here. |
Awesome! Thanks :) |
@bartlomiejbloniarz hi, after upgrading to 0.76 I faced the same issue on Paper, for some reason the modal won't show up only on iOS though. Can't give a reproducer since the code around showing the modal is kinda complex: an element is highlighted with:
Tried with different dialog components just in case react-native-ui-lib one is problematic, no luck, used rn Modal, no luck. edit: When I display the dialog without the reanimated svg rect the dialog shows up just fine 🤷♂️ edit2: under some unknown circumstances, when hot reloading the Dialog shows up, feels like it's one commit behind. |
@efstathiosntonas Could you open a separate issue for that? In this one the |
@chrfalch I am still working on a solution. I have a (seemingly) working approach, but I'm still not 100% sure if it won't break anything. The issue is that the non-visible content is actually rendered outside of the screen. This happens, because the Reanimated uses Reverting the changes of #6214 doesn't actually solve the problem, as the old algorithm has the exact same flaw (but only in Release mode). |
Thanks!! Really appreciate you working on this @bartlomiejbloniarz - I didn't catch the release issue - so this was a good find. |
I am not sure that I understand the conversation here, I am having this issue with Android only. My app is on RN 0.76.2 and all dialogs/modals/bottom sheets are broken as they either do not show up or they get stuck all smushed in the top left corner and the whole screen becomes unusable. Is there a fix for this or something I can patch to make it work for now? |
The only way I managed to fix this at is to just not have |
@bartlomiejbloniarz Hey there, would you mind sharing your possible solution? I have been taking a stab at getting this to work with no good progress. Figured it'd be a good idea to ask from someone with partial success. |
same problem |
Any updates on this issue? |
@bartlomiejbloniarz Checked #6776, issue with modals looks to be fixed. |
@bartlomiejbloniarz I can confirm that #6776 fixed issue for me too. |
@bartlomiejbloniarz I can confirm that #6776 fixes this issue for me |
The problem still persists. |
same for me, when update my react native app from o.75.4 to 0.76.5, the modal not working in android, since on ios it work perfectly |
Description
before Im upgrade to react native 0.76.1(new arch), Im using react native 0.75.4 and its work fine with old arch. I have trace error why my modal children is not rendered, because i have useAnimatedStyle in component where my modal placed.
When i disabled useAnimatedStyle, its not break modal
Steps to reproduce
Snack or a link to a repository
https://snack.expo.dev/@tamacroft_expo/2fd4f5
Reanimated version
3.16.1
React Native version
0.76.1
Platforms
Android
JavaScript runtime
Hermes
Workflow
React Native
Architecture
Fabric (New Architecture)
Build type
Debug app & production bundle
Device
Android emulator
Device model
No response
Acknowledgements
Yes
The text was updated successfully, but these errors were encountered: