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

fix(android)(8_0_X): TabGroup bar background color is wrongly transparent on Android 4.4 as of 8.0.0 #10930

Merged
merged 2 commits into from Jun 11, 2019

Conversation

jquick-axway
Copy link
Contributor

…transparent on Android 4.4

- Modified to fetch colors from ActionBar style XML for backward compaitibility with apps built before Titanium 8.0.0
@jquick-axway jquick-axway added this to the 8.0.2 milestone Jun 1, 2019
@jquick-axway jquick-axway requested a review from ypbnv June 1, 2019 00:43
@build build requested a review from a team June 1, 2019 01:07
@build
Copy link
Contributor

build commented Jun 1, 2019

Messages
📖

💾 Here's the generated SDK zipfile.

📖

✅ All tests are passing
Nice one! All 3033 tests are passing.

Generated by 🚫 dangerJS against 6ce0a26

@ssjsamir ssjsamir assigned ssjsamir and unassigned ssjsamir Jun 11, 2019
@ssjsamir ssjsamir self-requested a review June 11, 2019 14:15
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 TabGroup bar background color is no longer wrongly transparent on Android 4.4.

Tested using the test case mentioned in #10928 along with the test steps.

Test Environment

Nexus 5X (4.4.2)
7.0.12-70X.4
Operating System Name: Mac OS Mojave
Operating System Version: 10.14.2
Node.js Version: 10.13.0

@lokeshchdhry lokeshchdhry merged commit 16b2daf into tidev:8_0_X Jun 11, 2019
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

4 participants