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

[Bug]: The PB icon is not displayed for devices with Android <= 6 after setting a wallpaper #27366

Closed
AdinaPetridean opened this issue Oct 11, 2022 · 4 comments · Fixed by #27368 or nathanmkaya/fenix#108
Assignees
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:HomeScreen Feature:Wallpapers needs:triage Issue needs triage S3 Blocks non-critical functionality and a work around exists
Milestone

Comments

@AdinaPetridean
Copy link

AdinaPetridean commented Oct 11, 2022

Steps to reproduce

  1. Launch the Firefox app.
  2. Go to Settings- Homepage- Wallpapers and select a wallpaper.
  3. From the snackbar tap on "View" button.
  4. Observe that the PB icon is not displayed on homepage.

Expected behaviour

The PB icon is displayed on homepage when a wallpaper is set.

Actual behaviour

The PB icon is not displayed on devices with Android <= 6 after setting a wallpaper.
The icon is displayed for the default non-wallpaper option, when is set from Settings. However, note that when using the Wallpaper Selection Tool, the PB icon is not displayed, regardless of the wallpaper set, the default one included when the Light theme is used in combination with a previously set wallpaper on which the PB icon would be displayed in white color.

Device name

Huawei MediaPad M2

Android version

Android 5.1.1

Firefox release type

Firefox Nightly

Firefox version

Firefox Nightly 107.0a1 from 10/11

Device logs

No response

Additional information

This issue is also reproducible with Sony Xperia (Android 6.0.1). Not reproducible with Android >= 7.
Please note that the PB mode can be entered by tapping on the right corner, where the icon should be.
The issue seems to be a regression from #26644 in Nightly 107.0a1 from 09/27.
The issue was also uplifted to Beta 106 starting with Beta 3 and RC 106.1.0.

PBIcon_A5.mp4

┆Issue is synchronized with this Jira Task

@AdinaPetridean AdinaPetridean added 🐞 bug Crashes, Something isn't working, .. Feature:HomeScreen S3 Blocks non-critical functionality and a work around exists needs:triage Issue needs triage Feature:Wallpapers labels Oct 11, 2022
@Mugurell Mugurell self-assigned this Oct 11, 2022
Mugurell added a commit to Mugurell/fenix that referenced this issue Oct 11, 2022
…he new theme earlier to the private browsing button"

This reverts commit c7c5682 given that the
severity of the new issue is much higher.
@Mugurell
Copy link
Contributor

Seems like there is an issue with VectorDrawableCompat.create returning null on Android <= 23.

Given that the severity of this issue is much higher than of the one one for which the above call was added and that @Alexandru2909 is planning to soon refactor the private browsing button it seems like the best immediate solution would be to revert the #27105 patch, and also uplift this change.

@github-actions github-actions bot added the eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged label Oct 11, 2022
Mugurell added a commit to Mugurell/fenix that referenced this issue Oct 11, 2022
…he new theme earlier to the private browsing button"

This reverts commit c7c5682 given that the
severity of the new issue is much higher.
@MatthewTighe
Copy link
Contributor

Thanks for the report, and the fix!

We have decided not to uplift the patch for 106, given the following:

  • private browsing is still accessible through the tabs tray
  • the affected population should be relatively small (low number of users on versions that old, and of those only affects people with wallpapers)
  • today is already release candidate day

That said, prioritizing #27120 would be good for 107.

I'm also curious as to whether VectorDrawableCompat.create should've been caught by a linter. I'll investigate that a bit today

@mergify mergify bot closed this as completed in #27368 Oct 12, 2022
mergify bot pushed a commit that referenced this issue Oct 12, 2022
…private browsing button"

This reverts commit c7c5682 given that the
severity of the new issue is much higher.
@github-actions github-actions bot added this to the 107 milestone Oct 12, 2022
@github-actions github-actions bot reopened this Oct 12, 2022
@github-actions github-actions bot added eng:qa:needed QA Needed and removed eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged labels Oct 12, 2022
@delia-pop
Copy link

Verified as fixed on Nightly 107.0a1 from 10/13 with Huawei MediaPad M2 (Android 5.1.1). The PB icon is now visible with any wallpaper as well as with default option set after a wallpaper with white text.

20221013_141159_Trim.mp4

@wheelseal
Copy link

I have the same problem with my Lenovo Tablet with firefox 106.1. But I also have the FF-nightly installed (108a1) and see it is fixed.

JohanLorenzo pushed a commit to mozilla-mobile/firefox-android that referenced this issue Feb 14, 2023
…enix#26644 - Apply the new theme earlier to the private browsing button"

This reverts commit dbcd3fa given that the
severity of the new issue is much higher.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:HomeScreen Feature:Wallpapers needs:triage Issue needs triage S3 Blocks non-critical functionality and a work around exists
Projects
None yet
5 participants