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

Fixes an issue that appears when a block action causes another view to m... #62

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

giffnyc
Copy link

@giffnyc giffnyc commented Aug 27, 2013

...ake itself a subview of the UIApplication.keywindow. This bug was discovered when the Facebook API was called from a block action, displaying one of the custom FBDialog class windows. The FB API puts this view into the keywindow. When the FBDialog was dismissed, the BlockBackground remained, and made the underlying content unresponsive. This fix moves additional views in the BlockBackground view hierarchy out of itself when the time comes to give up keywindow status, and into the restored keywindow.

This fix may mess with fixes intended to handle multiple sheets to be called within one another and should be tested for this condition.

…o make itself a subview of the UIApplication.keywindow. This bug was discovered when the Facebook API was called from a block action, displaying one of the custom FBDialog class windows. The FB API puts this view into the keywindow. When the FBDialog was dismissed, the BlockBackground remained, and made the underlying content unresponsive. This fix moves additional view in the BlockBackground view hierarchy out of itself and into the restored keywindow.

This fix may mess with fixes intended to handle multiple sheets to be called within one another and should be tested for this condition.
@giffnyc giffnyc mentioned this pull request Aug 27, 2013
@barrettj
Copy link
Collaborator

Thanks - I'll try to take a look at these as soon as I'm done dealing with some iOS 7 issues.

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.

None yet

2 participants