-
Notifications
You must be signed in to change notification settings - Fork 987
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
fix(ios): autofocus input in drawer overlay won't trigger keyboard open #19964
Conversation
before: RPReplay_Final1715326577.MP4after (no idea why keyboard not visible in some screenshots): RPReplay_Final1715326458.MP4RPReplay_Final1715326433.MP4RPReplay_Final1715326396.MP4 |
Jenkins BuildsClick to see older builds (16)
|
83% of end-end tests have passed
Failed tests (7)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestDeepLinksOneDevice:
Class TestWalletOneDevice:
Class TestWalletMultipleDevice:
Class TestGroupChatMultipleDeviceMergedNewUI:
Expected to fail tests (2)Click to expandClass TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (43)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePR:
Class TestActivityMultipleDevicePRTwo:
Class TestWalletOneDevice:
Class TestGroupChatMultipleDeviceMergedNewUI:
|
on my iPhone 12 Pro, this is how it works, they keyboard still does not show RPReplay_Final1715338600.mov |
@yqrashawn are all the screenshots from the same device or is it that the keyboard is missing on some devices? |
@jo-mut they are from the same device, the keyboard is somehow not captured by the video, but it's actually showing up
I'll try reproduce |
93ff9e8
to
0a7e402
Compare
Hi @jo-mut, I tested on another device (iPad 17.4.1) and the keyboard shows up automatically, so can't reproduce on all my physical iOS devices |
0a7e402
to
96ca678
Compare
83% of end-end tests have passed
Failed tests (7)Click to expandClass TestDeepLinksOneDevice:
Class TestCommunityOneDeviceMerged:
Class TestWalletMultipleDevice:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestWalletOneDevice:
Expected to fail tests (2)Click to expandClass TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (43)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePRTwo:
Class TestWalletOneDevice:
|
Thanks for the fix @yqrashawn, PR can be merged. |
b82a738
to
8a5bc0c
Compare
8a5bc0c
to
1abd6da
Compare
fixes #19194
Summary
add
overlay.handleKeyboardEvents
opt toshow-overlay
before/after screenshots will be in the following comment, keyboard is not always visible in these videos
Review notes
react native navigation doc: https://wix.github.io/react-native-navigation/api/options-overlay/#handlekeyboardevents
Testing notes
this affects all usage of overlay (bottom sheet, toast, banner ...)
this only affects iOS platform
changes in this PR might impact some screens, and I'm not entirely sure if these are the desired outcomes. For instance, blur effects in below screen:
RPReplay_Final1715330782.MP4
Platforms
status: ready