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

Don't manipulate non-app windows #34

Open
RishatShamsutdinov opened this issue Aug 4, 2017 · 5 comments
Open

Don't manipulate non-app windows #34

RishatShamsutdinov opened this issue Aug 4, 2017 · 5 comments

Comments

@RishatShamsutdinov
Copy link

The lib always hides my non-app key window that I use for my custom controller in action sheet style that I use for photo picker.

UIWindow *keyWindow = LGAlertViewHelper.keyWindow;

[keyWindow endEditing:YES];

if (!hidden && keyWindow != LGAlertViewHelper.appWindow) {
    keyWindow.hidden = YES;
}
@darling0825
Copy link

/*
if (!hidden && keyWindow != LGAlertViewHelper.appWindow) {
keyWindow.hidden = YES;
}
*/

@NikKovIos
Copy link
Collaborator

NikKovIos commented Nov 23, 2017

When i have two windows. One with window?.windowLevel = UIWindowLevelNormal, other with UIWindowLevelNormal - 1. After

if (!hidden && keyWindow != LGAlertViewHelper.appWindow) {
        keyWindow.hidden = YES;
}

the screen becomes black. So i want to ask why is this necessary to hide a key window?

After some ivestigation this feature was added in d99e2ec
So if you have the same bug, you can use 2.3.0 version, it works well.

Fun stars count has this repo now =)
2017-11-23 21 48 13

@Friend-LGA
Copy link
Owner

Friend-LGA commented Mar 27, 2018

IIRC it needs to work properly together with other alert views. For example if you occasionally get some system alert view while showing LGAlertView.
You can check Alerts Cycle and Mix of Alerts options in demo project. If you have another ideas for workaround, it will be good to have it.

Please, attach some demo project with the reproducible issue. If I have time, I will take a look.

@RishatShamsutdinov
Copy link
Author

UIAlertController doesn't use separate window for presentation so hiding of key window doesn't resolve issues you mentioned.

@NikKovIos
Copy link
Collaborator

Also this line is causing a black screen appear (window hiding) in iPad.

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

4 participants