-
Notifications
You must be signed in to change notification settings - Fork 4.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
[RNMobile] Avoid navigation container dismissing the keyboard #53270
[RNMobile] Avoid navigation container dismissing the keyboard #53270
Conversation
I'm getting the following warning (seems I overlooked it because it was only being logged in the debugger 🤦).
Reverting the PR to draft as I noticed that UPDATE: Fixed in d4bdfc7. |
Size Change: 0 B Total Size: 1.44 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for fixing this 🚀
What?
Disallowing navigation containers to control the keyboard and dismiss it when navigation occurs.
Why?
This is a regression introduced when updating the React Navigation library via the React Native upgrade 0.71.11.
How?
Pass the prop
keyboardHandlingEnabled
to the navigation containers with valuefalse
. This should be enough to address the regression, based on the documentation:Testing Instructions
Testing Instructions for Keyboard
N/A
Screenshots or screencast
ios-focus-after-dismissing-bottom-sheet.mp4