Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

[android 5] The insecure lock icon is white on the toolbar #6022

Closed
lobontiumira opened this issue Dec 9, 2021 · 6 comments
Closed

[android 5] The insecure lock icon is white on the toolbar #6022

lobontiumira opened this issue Dec 9, 2021 · 6 comments
Assignees
Milestone

Comments

@lobontiumira
Copy link

Steps to reproduce

  1. On an Android 5 devices, install Focus, and set the Light theme.
  2. Open an insecure page.
  3. Observe the insecure lock icon from the toolbar.

Expected behavior

The insecure lock icon is visible.

Actual behavior

The insecure lock icon is white, and not visible.

Device information

  • Android device: Samsung Galaxy Tab a6 (Android 5.1.1), Xiaomi mi4i (Android 5.0.2)
  • Focus version: Beta 96.0.0-beta.1
  • Not reproducible on Focus Nightly 95.0 (353421248 GV 96.0a1-20211204095400)

white icon

@ionutbedregeanu ionutbedregeanu self-assigned this Jan 11, 2022
@ionutbedregeanu
Copy link
Contributor

@lobontiumira Still reproducing? I tried on Pixel 5 Android 5.1 and I didn't succeed

@lobontiumira
Copy link
Author

Yes, I am still able to reproduce this issue on Xiaomi mi4i (Android 5.0.2), on Focus Nightly 97.0 (build 360070846 GV 97.0a1-20220106155649):

Screenshot_2022-01-11-15-49-50-658_org mozilla focus nightly

@ionutbedregeanu
Copy link
Contributor

mozilla-mobile/android-components#11537 should fix the bug

@Mugurell
Copy link
Contributor

Looked into this more with Ionut and Mira and saw that this issue happens only after closing a tab, not with the first tab opened from a cold start and this seems like an intermittent issue, with cases in which this will work even in this scenario.

This seems to all depend on the fact that when a tab is closed we are resetting (clearing) the list of indicator and colors used for them and then seemingly there is an edgecase in which these are not properly updated when loading a new page.

@ionutbedregeanu
Copy link
Contributor

To validate the fix we need to wait mozilla-mobile/android-components#11546 to be merged

@ionutbedregeanu ionutbedregeanu added the QAReady flag for test read & needed label Jan 26, 2022
@delia-pop
Copy link

delia-pop commented Jan 28, 2022

This is fixed on Focus Nightly 98.0 (build 360280510 GV 98.0a1-20220127094620) with Huawei MediaPad M2 (Android 5.1.1).

Screenshot_2022-01-28-15-26-33

Note that is still present in Beta 97.0.0-beta.6.

@delia-pop delia-pop added QAVerified and removed QAReady flag for test read & needed labels Jan 31, 2022
@ionutbedregeanu ionutbedregeanu added this to the 98 milestone Feb 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants