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-24805] (6_3_X) iOS11: Support large window titles #9431

Merged
merged 12 commits into from Oct 11, 2017

Conversation

hansemannn
Copy link
Collaborator

Copy link
Contributor

@vijaysingh-axway vijaysingh-axway left a comment

Choose a reason for hiding this comment

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

CR Passed.
Note for QE - Please verify it after merging #9430 , as IS_XCODE_9 macro is defined there.

@ssjsamir ssjsamir self-requested a review October 11, 2017 17:01
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 a large window title when largeTitleDisplayMode: Ti.UI.iOS.LARGE_TITLE_DISPLAY_MODE_ALWAYSwas used.

Test Steps:

  • Created a new titanium project
  • Added the code from the description in to the project
  • Ran the application from studio
  • Saw window had a large title
    largeTitleDisplayMode: Ti.UI.iOS.LARGE_TITLE_DISPLAY_MODE_ALWAYS

screen shot 2017-10-11 at 18 00 31

+ Set `largeTitleDisplayMode: Ti.UI.iOS.LARGE_TITLE_DISPLAY_MODE_ALWAYS` to `Ti.UI.iOS.LARGE_TITLE_DISPLAY_MODE_NEVER`
  • Saw that title was no longer large

screen shot 2017-10-11 at 18 01 29

Test Environment
Appcelerator Command-Line Interface, version 6.2.4
iPhone 7 emulator (11.0)
iphone 5S plus 11.0.3
Operating System Name: Mac OS High Sierra
Operating System Version: 10.13
Node.js Version: 6.10.1
Xcode: 9.0
Appcelerator Studio: 4.10.0.201709271713

@ssjsamir ssjsamir merged commit 4dc9f7b into tidev:6_3_X Oct 11, 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