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-24799] Fix issue where theme was being set as its parent #9134

Merged
merged 2 commits into from Jun 12, 2017

Conversation

ewanharris
Copy link
Collaborator

@ewanharris ewanharris commented Jun 9, 2017

JIRA: https://jira.appcelerator.org/browse/TIMOB-24799

Bad things happen if a theme is a parent of itself, so we should prevent that

Test steps in JIRA

@ewanharris
Copy link
Collaborator Author

Eurgh, I just realised I inadvertently pushed to appcelerator/titanium_mobile not my fork. @sgtcoolguy will that screw things up at all? Should I delete the branch here and redo from my fork?

Copy link
Contributor

@garymathews garymathews left a comment

Choose a reason for hiding this comment

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

CR: PASS

@lokeshchdhry
Copy link
Contributor

FR Passed.

Setting theme as <application android:theme="@style/Theme.Titanium"/> in the tiapp.xml does not inherit itself & no error seen.

  1. Ran the attached app in the ticket with the fix & no error/crash was seen.

Studio Ver: 4.9.0.201705302345
SDK Ver: 6.2.0 local build
OS Ver: 10.12.3
Xcode Ver: Xcode 8.3.3
Appc NPM: 4.2.9
Appc CLI: 6.2.2
Ti CLI Ver: 5.0.14
Alloy Ver: 1.9.11
Node Ver: 6.10.1
Java Ver: 1.8.0_101
Devices: ⇨ google Nexus 5 --- Android 6.0.1
⇨ google Pixel --- Android 7.1.1
Emulator: Android 4.4.2

@lokeshchdhry lokeshchdhry merged commit a7b2ac8 into master Jun 12, 2017
@ewanharris ewanharris deleted the TIMOB-24799 branch June 12, 2017 22:53
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