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

unexpected action sheet dismissing in portrait mode only #40

Merged
merged 1 commit into from Sep 17, 2014
Merged

unexpected action sheet dismissing in portrait mode only #40

merged 1 commit into from Sep 17, 2014

Conversation

numen31337
Copy link

currently if my application support only portrait mode and I rotate the device, picker is dismissing unexpectedly

@ghost
Copy link

ghost commented Sep 17, 2014

The problem with that code is that it only works for portrait to landscape. What about the reverse?

@skywinder
Copy link
Owner

@numen31337 Your're right. But before merge I would like to improve your code to support not only UIInterfaceOrientationMaskPortrait case, but any other combinations of supported masks.

For example: your code doesn't work in case when I set supported device orientation to Landscape left.

@numen31337
Copy link
Author

yep, you are right, sorry for fix only for my case, I was so wondering why it disappeared while I'm selecting :)

@skywinder skywinder merged commit 31b7266 into skywinder:master Sep 17, 2014
skywinder added a commit that referenced this pull request Sep 17, 2014
@skywinder
Copy link
Owner

@numen31337 I updated your code. Now it works for all orientations.

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