iOS screen is frozen when swiping to previous page

[joonhocho]

Description

Original issue: react-navigation/react-navigation#9015
iOS screen is frozen when swiping to previous page when detachInactiveScreens is true.
https://reactnavigation.org/docs/stack-navigator/#detachpreviousscreen

Screenshots

NA

Steps To Reproduce

1. On iOS simulator, with react navigation stacks, try swipe back to previous screen
   when having a deep stack of screens (at least 3 or 4 levels deep)

2. App freezes completely

Expected behavior

It should not freeze and swipe back to previous screen

Actual behavior

It freezes completely

Snack or minimal code example

NA

Package versions

• React: 17.0.1
• React Native: 0.63.3
• React Native Screens: 2.15.0
• @react-navigation/native: 5.8.10

[WoLewicki]

Can you provide a repo/snack with minimal configuration needed to reproduce the issue?

[todorone]

@WoLewicki That's the minimal reproduction - https://snack.expo.io/@todorone/697343
When trying to swipe back from second screen to first, it immediately freezes(iOS only). If keyboard is hidden or enableScreens() is not called, no freeze occurs.
Relevant for the latest version of react-native-screens.

[WoLewicki]

Thank you @todorone! I reproduced it and it happens due to keyboardHandlingEnabled={false} and it looks like a bug on the native iOS side. When swiping to change screen, we disable user interaction on the native side, which makes the textinput fire an event textInputDidEndEditing which wants to aquire the same mutex as the swiping gesture, so it leads to a deadlock. When keyboard dismissing is handled by react-navigation, the dismissing event is fired before the user interaction disabling, so it works fine.
The only workaround for now is to set keyboardHandlingEnabled={true}.

[todorone]

@WoLewicki Great, happy that reproduction helped. Unfortunately, enabling keyboardHandlingEnabled is not an option for us, so we will patiently wait for fix...🐱

[github-actions]

Issue validator - update # 1

Hello!
Congratulations! Your issue passed the validator! Thank you!

[WoLewicki]

Can you check if #777 resolves the issue and does not introduce any new bugs? It may have some effects that we do not have knowledge of, so it would extremely helpful to test it in many examples.

[todorone]

@WoLewicki Just tried, works perfect for me. Will report on side effects if any...

[hunterbmt]

This also happens when swiping back after moving the app to the background, could you help to confirm the fix on this case as well @WoLewicki

[WoLewicki]

@hunterbmt I am not sure what flow do you mean exactly. Moving app to the background, opening it again and then swiping? If so, I tested it and there is no issue then, but I don't think there should be any regarding it. You can always fork the repo or add the changes to your project and check it by yourself, it would be very helpful.