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

[TIMOB-24496]:iOS Camera overlay does not fill the entire screen when orientation is changed on iPad, Titanium SDK 6.0.0 and later #8915

Merged
merged 1 commit into from Mar 30, 2017

Conversation

vijaysingh-axway
Copy link
Contributor

… orientation is changed on iPad, Titanium SDK 6.0.0 and later
@hansemannn
Copy link
Collaborator

hansemannn commented Mar 29, 2017

CR passed, the change I made was unnecessary!

@htbryant94 htbryant94 requested review from htbryant94 and removed request for htbryant94 March 29, 2017 19:33
@ssjsamir ssjsamir self-requested a review March 30, 2017 17:21
Copy link
Contributor

@ssjsamir ssjsamir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FR Passed, was able to see the entire camera overlay when the screen orientation of an iPad was set to landscape.

Test Steps

  • Built the SDK from this PR
  • Copied the code in the description of the Jira ticket (https://jira.appcelerator.org/browse/TIMOB-24496)
  • Pasted the code in to a titanium project
  • Ran the program
  • Saw the camera overlay when the application first launched (Portrait)
  • Changed the orientation of the iPad to landscape and saw that the overlay was fully visible

I then re-ran the steps on an older SDK 6.0.2.GA and saw that the overlay was not visible in landscape mode thus verifying that changes were made.

Environment
Appcelerator Command-Line Interface, version 6.1.0
Apple iPad air 2 (iOS 9.2)
Operating System Name: Mac OS X El Capitan
Operating System Version: 10.11.6
Node.js Version: 4.6.0
npm: 4.2.8
Xcode: 8.2
Appcelerator Studio: 4.8.1.201612050850

@ssjsamir ssjsamir merged commit 6d7c29d into tidev:master Mar 30, 2017
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

3 participants