-
Notifications
You must be signed in to change notification settings - Fork 268
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
Problem with FBDialog #32
Comments
Can observe the same problem - I can tap through BlockAlert or BlockActionSheet and can't close/dismiss them. |
This may have been fixed by a previous commit, let me know if there are still issues as I don't use FBDialog. |
This issue should stay open as I confirm it still persists. It's also fairly easy to reproduce. If you open a FBDialog from the alert's button block, the alert is dismissed (animation wise) but the screen is still dimmed and locked (UI unresponsive) |
Reopened; but as none of the maintainers use FBDialog a fix from one of us is unlikely. We'll gladly welcome a pull--request that fixes the issue. |
A pull request was submitted. The issue appears to be that the FBDialog class makes itself a child of the UIApplication.keywindow. When called within a blockaction, this FBDIalog view remains within the BlockBackground view hierarchy, and prevents the view from resigning its role as keywindow. The fix moves such views into the previous keywindow so that it can be restored. This was a quicky fix - @Barretj and others are probably more qualified than I to asses unintended consequences of this fix. But it works in my app tests. |
When a FBDialog view is displayed by pressing a button in the actionsheet, the actionsheet does not dismiss correctly: the UI does not respond to touches.
The text was updated successfully, but these errors were encountered: