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): TabGroup bar background color is wrongly transparent on Android 4.4 as of 8.0.0 #10928

Merged
merged 3 commits into from Jun 11, 2019

Conversation

jquick-axway
Copy link
Contributor

@jquick-axway jquick-axway commented May 31, 2019

JIRA:

Summary:

  • TIMOB-27117 Fixed bug where TabGroup bar background color is wrongly transparent on Android 4.4 as of Titanium 8.0.0.
  • TIMOB-27100 Fixed 8.1.0 regression where tab text color is wrongly transparent.
    • This was bug was never in a released version. Caught before release.
  • Modified TabGroup to fetch bar background color and text color from ActionBar style/theme for backward compatibility.

Test:

  1. Set up a Classic Titanium app with the code attached to TIMOB-27117.
  2. Build and run on an Android 4.4 device.
  3. Verify that the tab labels appear.
  4. Verify that the tab bar's background is dark gray like the title bar.

…transparent on Android 4.4

- Regression as of Titanium 8.0.0
- 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.1.0 milestone May 31, 2019
@jquick-axway jquick-axway requested a review from ypbnv May 31, 2019 18:41
@build build requested a review from a team May 31, 2019 18:58
@build
Copy link
Contributor

build commented May 31, 2019

Messages
📖

💾 Here's the generated SDK zipfile.

📖

✅ All tests are passing
Nice one! All 3613 tests are passing.
(There are 464 tests skipped)

Generated by 🚫 dangerJS against bf9f78b

Copy link
Contributor

@ypbnv ypbnv left a comment

Choose a reason for hiding this comment

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

LGTM!

@ssjsamir ssjsamir self-requested a review June 11, 2019 14:08
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 above 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 54afbc7 into tidev:master Jun 11, 2019
hansemannn pushed a commit to hansemannn/titanium_mobile that referenced this pull request Jul 8, 2019
…transparent on Android 4.4 (tidev#10928)

- Regression as of Titanium 8.0.0
- Modified to fetch colors from ActionBar style XML for backward compaitibility with apps built before Titanium 8.0.0
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

6 participants