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

[Bug]: The Wallpaper Selection Tool is not displayed on homepage #26954

Closed
delia-pop opened this issue Sep 13, 2022 · 8 comments
Closed

[Bug]: The Wallpaper Selection Tool is not displayed on homepage #26954

delia-pop opened this issue Sep 13, 2022 · 8 comments
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:Wallpapers S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist

Comments

@delia-pop
Copy link

delia-pop commented Sep 13, 2022

Steps to reproduce

  1. Launch Fenix.
  2. Dismiss the onboarding.
  3. Visit a webpage and then return to homepage. Repeat this 2 more times.
  4. Observe if the Wallpaper Selection Tool is displayed on homepage.

Expected behaviour

The wallpaper selection tool is displayed on the third homepage visit.

As per requirements:
A wallpaper selection tool will be displayed on the homepage under the following the conditions:

  • Tab Pickup and Jump Back In CFRs are not currently being displayed
  • Wallpaper thumbnails have been downloaded
  • The user has not seen the wallpaper selection tool before

Actual behaviour

The Wallpaper Selection Tool is not displayed on homepage.
Note that the wallpapers were fully downloaded.

Device name

Google Pixel 6

Android version

Android 13

Firefox release type

Firefox Nightly

Firefox version

Nightly 106 from 09/13/Debug build updated from main

Device logs

No response

Additional information

No response

┆Issue is synchronized with this Jira Task

@delia-pop delia-pop added 🐞 bug Crashes, Something isn't working, .. S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist needs:triage Issue needs triage Feature:Wallpapers labels Sep 13, 2022
@delia-pop delia-pop changed the title [Bug]: The Wallpaper Selection Tool is not displayed o homepage [Bug]: The Wallpaper Selection Tool is not displayed on homepage Sep 13, 2022
@Mugurell
Copy link
Contributor

Not sure the new tool is intended to be available in Nightly as it currently seems to be available to just debug but even in debug I see some potential issues when investigating this:

@mavduevskiy

@amedyne amedyne removed the needs:triage Issue needs triage label Sep 13, 2022
@MatthewTighe
Copy link
Contributor

I am curious if you're still seeing this issue, and for which wallpaper. The dreamer thumbnail was previously missing but has since been added and should work now.

we wait until the jump back in cfr is shown. Seems like rather this tool should be managed also by the HomeCFRPresenter and not wait for other cfrs if it's conditions to be shown are met.

The intent is to wait until all CFRs have been shown before showing the wallpaper selection tool. That said, I think there is a follow-up opportunity to move all our logic around when onboarding is shown into a single class like the HomeCFRPresenter.

@MatthewTighe
Copy link
Contributor

It's worth noting that the S3 bucket that hosts the wallpaper assets can take a while to refresh regional caches. I fixed the dreamer issue around 10 or 11 PST yesterday (about 24 hours before this comment) but we have seen it take a while for updates to propagate. It will be good to plan to update the remote server at least 48 hours in advance of any required deadlines.

@Mugurell
Copy link
Contributor

I am curious if you're still seeing this issue, and for which wallpaper. The dreamer thumbnail was previously missing but has since been added and should work now.

Yesterday and today I'm seeing the same wallpaper with problems - innovator.
Seems like it will stop causing issues after #26971

@MatthewTighe
Copy link
Contributor

Ah, looks like I actually forgot to add the assets for that one. Thanks for mentioning it! The update might take a while to propagate to the servers but they should be available soon

@delia-pop
Copy link
Author

Hi, @MatthewTighe, on the latest Nightly 107.0a1 from 09/19 the wallpaper thumbnails from both Classic and Limited Edition collections are displayed and the wallpapers can be properly downloaded and set. Should the wallpaper selection tool also be available in Nightly any time soon?

@MatthewTighe
Copy link
Contributor

It looks like we forgot to enable the feature flag 😅 We will land something today that should enable it in the next nightly

@delia-pop
Copy link
Author

Verified as fixed on Nightly 107.0a1 from 09/21 with Google Pixel 6 (Android 13). The wallpaper selection tool is displayed on the 3rd homepage visit.

1663766721143

@delia-pop delia-pop added the eng:qa:verified QA Verified label Sep 21, 2022
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:Wallpapers S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist
Projects
None yet
Development

No branches or pull requests

4 participants