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

Avoid stealing touches from other native modals. #47

Merged
merged 1 commit into from
Aug 30, 2019

Conversation

danielgindi
Copy link
Contributor

With react-native-navigation, an overlay with interceptTouchesOutside=false is not allowed to pass through the touches because of the wrapper view introduces by react-native-root-siblings.
As I've seen in the issues section in react-native-navigation, many people ditch libraries like react-native-popup-dialog and react-native-root-toast due to this issue.

With `react-native-navigation`, an overlay with `interceptTouchesOutside=false` is not allowed to pass through the touches because of the wrapper view introduces by `react-native-root-siblings`.
As I've seen in the issues section in `react-native-navigation`, many people ditch libraries like `react-native-popup-dialog` and `react-native-root-toast` due to this issue.
@danielgindi
Copy link
Contributor Author

@magicismight This is an important one :-)

@sunnylqm sunnylqm merged commit 7cd2e09 into magicismight:master Aug 30, 2019
@danielgindi
Copy link
Contributor Author

Care for a release? :-)

@sunnylqm
Copy link
Collaborator

sunnylqm commented Sep 6, 2019

@danielgindi I remember I already published?

@danielgindi
Copy link
Contributor Author

Oh yes. Seems like you have published to npm but no commit/tag here with the version, so I got confused. Sorry!

@sunnylqm
Copy link
Collaborator

sunnylqm commented Sep 6, 2019

@danielgindi Sorry not that formal 🤣

micheleb added a commit to urbi-mobility/react-native-formik that referenced this pull request Apr 28, 2020
Include this PR that fixes touch events on react-native-navigation
overlays:
magicismight/react-native-root-siblings#47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants