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

dismissOnHardwareBackPress = false and dismissOnTouchOutside = false not working #2

Open
boliveira opened this issue May 23, 2017 · 4 comments

Comments

@boliveira
Copy link

Hi,

I tried to show a dialog with DialogManager.show with dismissOnHardwareBackPress = false and dismissOnTouchOutside = false and it's not working, it's dismissing anyway.
Took a look at the code and the dismissOnHardwareBackPress is not passed to PopupDialog instance on DialogComponent render, and on Dialog.js there is a property closeOnTouchOutside that exists for backwards compatibility but it's value is by default true and it ignores the dismissOnTouchOutside value.

@jacklam718
Copy link
Owner

@boliveira
Did you set haveOverlay = false? because if haveOverlay is false then dismissOnTouchOutside won't work.

@boliveira
Copy link
Author

No, haveOverlay is true, I checked that. I ended up changing Dialog.js code to remove the close*** props and changed DialogComponent.js to pass dismissOnHardwareBackPress to PopupDialog and then it worked.

@jacklam718
Copy link
Owner

@boliveira Can you try version v0.1.11 thanks. if there are any problem please let me know. Thanks a lot.

@boliveira
Copy link
Author

@jacklam718 , tested version v0.1.11 and it works now. Thank you very much!

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

No branches or pull requests

2 participants