Skip to content
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

Mobile: Fixes #6682: Disable KeyboardAvoidingView when floating keyboard is visible #7593

Conversation

personalizedrefrigerator
Copy link
Collaborator

@personalizedrefrigerator personalizedrefrigerator commented Jan 7, 2023

Summary

Testing plan

  1. If on an iPad, enable the floating keyboard.
  2. Open the note search view and search for a note (using software keyboard).
  3. Edit a note (using software keyboard)
  4. Hide the software keyboard.
  5. Repeat steps 1-4 with floating keyboard disabled.

To-do

  • Test on Android to ensure this doesn't break anything

@laurent22
Copy link
Owner

Maybe we should just upgrade RN?

@personalizedrefrigerator
Copy link
Collaborator Author

personalizedrefrigerator commented Jan 7, 2023

Maybe we should just upgrade RN?

That's definitely worth trying. The fix in React Native 0.70.x is conditioned on prefersCrossFadeTransitions() returning true, however. Thus, it's unclear whether upgrading will also fix the floating keyboard issue:

facebook/react-native@4b9382c#diff-faf166615ccb0139ebe90f2db2de3e248abf7c11c8581e7223db69658cf5f87dR88

Edit: The upstream floating keyboard issue is still open.

@laurent22
Copy link
Owner

Are you able to replicate this bug on simulator? I've just upgrade to RN 0.70 so it might be worth checking if you know how.

Otherwise we can indeed add the patch, but it seems to have a dependency to a specific RN version? (0.68.5)

@personalizedrefrigerator personalizedrefrigerator changed the title Mobile: Fixes #6682: Patch react-native to avoid KeyboardAvoidingView collapse Mobile: Fixes #6682: Disable KeyboardAvoidingView when floating keyboard is visible Jan 8, 2023
@personalizedrefrigerator
Copy link
Collaborator Author

Are you able to replicate this bug on simulator? I've just upgrade to RN 0.70 so it might be worth checking if you know how.

Otherwise we can indeed add the patch, but it seems to have a dependency to a specific RN version? (0.68.5)

With RN 0.70, I can't replicate the crossfade-transition-related bug, but can still replicate the floating keyboard bug. I've updated the PR to apply the fix recommended in the comments of the bug report for the floating-keyboard-related issue.

@laurent22
Copy link
Owner

Looks good, thanks for fixing this!

@laurent22 laurent22 merged commit b695acf into laurent22:dev Jan 11, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jan 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[iOS] App must be restarted after each note edit
2 participants