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

CB-13193: (ios) Fixed Lock iOS Landscape Orientation turn up-sidedown #25

Merged

Conversation

fortunes-technology
Copy link
Contributor

Platforms affected

iOS

What does this PR do?

In iOS, the Landscape have opposite orientation.
Meaning if you do below when you are in landscape orientation, then you will notice the screen flips to opposite landscape orientation.
screen.orientation.lock(screen.orientation.type);

What testing has been done on this change?

Manual testing on iPhone has been done.

@fpirsch
Copy link

fpirsch commented Jan 9, 2018

👍

@jfbloom22
Copy link

Thanks! Verified this fixes the issue for me on iOS 11.2.2 iPad mini 4.

@whodeee
Copy link

whodeee commented Jan 31, 2018

Any chance you can get the plugin updated, when getting it using cordova CLI, it is not showing this update.

@procom
Copy link

procom commented Feb 6, 2018

Hello,

+1 @whodeee

These fixes are not integrate in the release

@fortunes-technology
Copy link
Contributor Author

The Merge conflict fixed.
This PR fixes, the landscape right and left issue.
Also if you want to lock orientation to landscape (do not care when it is right or left) and it's already in landscape mode (whether it is right or left), we shouldn't be changing orientation.
Right now, if we change orientation to landscape, it just change it to landscape left.
So this is fixed with this PR.

@ptorrent
Copy link

ptorrent commented Mar 6, 2018

Hello,

I don't understand how works this plugin..

If I do screen.orientation.lock('portrait') , it changes nothing.
screen.orientation.lock('landscape') too..
I can always change the orientation of my screen.

If I close the application and go back inside , it's works!

Thanks for your reply

@SteveEdson
Copy link

Please can this be merged? I'm also experiencing this issue and need the fix. Thanks!

@davidofwatkins
Copy link

I can confirm this works for me as well (iPhone 5, iOS 10.3.3).

@fortunes-technology
Copy link
Contributor Author

@macdonst @purplecabbage @stevengill
Can someone review this PR please?

@jcesarmobile jcesarmobile merged commit 26eb47e into apache:master Jul 1, 2018
@jcesarmobile jcesarmobile mentioned this pull request Jul 1, 2018
3 tasks
@sbatezat
Copy link

Any chance to see this PR released in a tag?

@jfougere
Copy link

Any news of a 3.0.2 release ?

@ghost
Copy link

ghost commented Nov 12, 2018

@maverickmishra do you have any insight into whether a 3.0.2 release of this plugin is planned? If not, do you know who would? Thank you.

@davidofwatkins
Copy link

+1, any news here about getting this into an official release @jcesarmobile?

@janpio
Copy link
Member

janpio commented Jul 4, 2019

3.0.2 has been released and contains this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet