1406329 - (photon) Tablet - After removing a theme, the tab highlight remains while switching tabs

Status: RESOLVED INCOMPLETE

(Firefox for Android Graveyard :: Theme and Visual Design, defect, P2)

Description

[Oana Horvath [:ohorvath - out of office]]

Attachment: recording1.mp4

Device:
Huawei MediaPad M2 (Android 5.1.1)
Galaxy Tab S3 (Android 7.0)

Build: Beta 57.0b5 (2017-10-03);

Steps to reproduce:
1. Add a lightweight theme.
2. Open a few tabs.
3. Remove the theme and open tabs and scroll the tab strip.

Expected result:
The tabs contrasting background should be removed completely with the theme.

Actual result:
The tab semi-transparent background that makes the tab titles visible when a theme is applied, is randomly applied on tabs after the theme was removed.
See video attached.

Comment 1

[Marcia Knous [:marcia]]

Not able to reproduce this yet on a Nexus 7 tablet using the latest beta 5 and running Android 6.0.1.

Comment 2

[Liz Henry (:lizzard) (relman/hg->git project)]

This doesn't sound like a release blocker to me - a bit of an edge case. We could aim for a fix for 58 if the bug is still reproducible. 
Anthony, what do you think?

Comment 3

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Hm... that's weird. This is definitely a bug that we should fix. 

But given 57's deadline, and the repro issues (I also can't repro it) I don't think this is necessarily a blocker for 57. 

Ioan, do you have any luck repro-ing this issue? :(

Comment 4

[Marcia Knous [:marcia]]

We will wait to see how reproducible this is before deciding whether to track it for 58.

Comment 5

[Jeff Griffiths (:canuckistani) (:⚡︎)]

(In reply to Anthony Lam (:antlam) from comment #3)
...
> But given 57's deadline, and the repro issues (I also can't repro it) I
> don't think this is necessarily a blocker for 57. 

Agreed, based on, well, assumption about how common theme usage is. I assume it's relatively low.

Comment 6

[Wesly Huang (EPM)]

Sorry I should have marked it P2 after seeing Liz's comment last week (and I agree it's not a 57 blocker). Also given the team's current loading and focus it's less likely to get it in 58's Nightly cycle per my understanding.

Comment 7

[Ioana Chiorean]

(In reply to Anthony Lam (:antlam) from comment #3)
> Hm... that's weird. This is definitely a bug that we should fix. 
> 
> But given 57's deadline, and the repro issues (I also can't repro it) I
> don't think this is necessarily a blocker for 57. 
> 
> Ioan, do you have any luck repro-ing this issue? :(

Was this addressed to me? Ioan Chiciuden is no longer in our team ( not sure is he is mozilling around after )

Comment 8

[BMO Automation]

We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.