Closed Bug 1737770 Opened 1 year ago Closed 1 year ago

[Experiment] The Firefox logo icon is wrongly displayed instead of the "umbrella" one on the "emotive-pin-firefox-modal" from the "Treatment B" branch

Categories

(Firefox :: Messaging System, defect, P1)

Desktop
Windows
defect

Tracking

()

VERIFIED FIXED
95 Branch
Iteration:
95.2 - Oct 18 - Oct 31
Tracking Status
firefox93 --- wontfix
firefox94 + verified
firefox95 + verified

People

(Reporter: mcoman, Assigned: Mardak)

References

(Blocks 2 open bugs)

Details

Attachments

(2 files)

Attached image ss of the issue.png

[Affected versions]:

  • Firefox Release 93.0 - Build ID: 20210927210923
  • Firefox Beta 94.0 - Build ID: 20211025220926

[Affected Platforms]:

  • Windows 10 x64

[Prerequisites]:

  • Have a Firefox profile older than 28 days.
  • Have this user.js file saved to your PC.
  • Have at least 1 URL accessed in the last 13 days.
  • The Firefox browser is not pinned to the taskbar.

[Steps to reproduce]:

  1. Navigate to the profile folder from prerequisites and paste the "user.js" file.
  2. Open the browser with the profile from prerequisites.
  3. Observe the icon from the "Pin to taskbar" modal.

[Expected result]:

  • An "umbrella" icon is displayed in the top part of the modal.

[Actual result]:

  • The Firefox logo is displayed instead.

[Notes]:

  • Attached a screenshot of the issue.

We're expecting the image to become available at https://www.mozilla.org/media/img/firefox/onboarding/umbrella.041dbea876c4.png as added in https://github.com/mozilla/bedrock/issues/10632 but it has not been deployed to production yet.

The image is now on stage. I'll update nimbus recipe when it gets to prod.

https://www.allizom.org/media/img/firefox/onboarding/umbrella.041dbea876c4.png

Assignee: nobody → edilee
Iteration: --- → 95.2 - Oct 18 - Oct 31
Priority: -- → P1

Doh. Recipe is updated but content security on the modal prevents loading https or data URIs.

Content Security Policy: The page’s settings blocked the loading of a resource at https://www.mozilla.org/media/img/firefox/onboarding/umbrella.041dbea876c4.png (“default-src”).

[Tracking Requested - why for this release]: Would like to ride along to 94 to show a more emotive image than just the firefox logo. If not for this specific experiment, there are others planned to run in November.

Pushed by elee@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f38bcae5f766
Allow spotlight messaging modal to show images from mozilla r=andreio

Marius, seems like we'll use the Firefox logo in treatment b to launch the experiment now. But we're fixing this bug to allow umbrella and other images for future experiments in 95 and potentially 94 if uplifted. As of right now, emotive-pin-firefox-modal still is configured to try to show umbrella for treatment b, so if this code change lands, you could verify that umbrella shows up in nightly.

But depending on when we actually launch it, the image will be switched to firefox logo. So to verify, we'll need a different experiment/preview that uses the umbrella, so maybe I'll make a clone of the experiment in preparation for a separate upcoming experiment (sounds like there's at least 2 more planned for November).

Here's a new experiment on preview that currently uses the umbrella (only 1/control branch):
https://experimenter.services.mozilla.com/nimbus/peace-of-mind-casual-modal

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch

Comment on attachment 9247884 [details]
Bug 1737770 - Allow spotlight messaging modal to show images from mozilla r=andreio a?release

Beta/Release Uplift Approval Request

  • User impact if declined: Emotive messaging experiments are less exciting with a generic Firefox logo
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: Yes comment 13
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: 1. Enroll into https://experimenter.services.mozilla.com/nimbus/peace-of-mind-casual-modal
    a. Set nimbus.debug to true
    b. Navigate to about:studies?optin_slug=peace-of-mind-casual-modal&optin_branch=control&optin_collection=nimbus-preview
  1. Force show peace-of-mind-casual-modal message
    a. Set browser.newtabpage.activity-stream.asrouter.devtoolsEnabled to true
    b. Open new tab
    c. Click devtools wrench in top right
    d. Find in page "peace-of-mind-casual-modal"
    e. Click "Show" for the message
    (Might need to restart Firefox if already opened a new tab and message doesn't show up.)
Attachment #9247884 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Flags: needinfo?(mcoman)

Noticed there's a beta code conflict with bug 1733623, which only landed on 95. I'll update the patch for beta.

Comment on attachment 9247884 [details]
Bug 1737770 - Allow spotlight messaging modal to show images from mozilla r=andreio a?release

94 is on release now.

Attachment #9247884 - Flags: approval-mozilla-beta? → approval-mozilla-release?
Attachment #9247884 - Attachment description: Bug 1737770 - Allow spotlight messaging modal to show images from mozilla r?andreio → Bug 1737770 - Allow spotlight messaging modal to show images from mozilla r=andreio a?release
QA Whiteboard: [qa-triaged]

I have verified that this issue is no longer reproducible, using the steps that Ed provided in comment 10, with the latest Firefox Nightly (95.0a1 Build ID - 20211027214117) installed on Windows 10 x64, macOS 11.6, and Linux Mint 20.2 x64. Now, I can confirm that the "umbrella" logo is successfully displayed in the top part of the "peace-of-mind-casual-modal".

Status: RESOLVED → VERIFIED
Flags: needinfo?(mcoman)

Comment on attachment 9247884 [details]
Bug 1737770 - Allow spotlight messaging modal to show images from mozilla r=andreio a?release

Approved for 94.0rc2.

Attachment #9247884 - Flags: approval-mozilla-release? → approval-mozilla-release+

I have verified that this issue is no longer reproducible, using the steps that Ed provided in comment 10, with the latest Firefox Beta RC (94.0 Build ID - 20211028161635) installed on Windows 10 x64, macOS 11.6, and Linux Mint 20.2 x64. Now, I can confirm that the "umbrella" logo is successfully displayed in the top part of the "peace-of-mind-casual-modal".

Flags: qe-verify+
Blocks: 1739252
See Also: → 1765411
You need to log in before you can comment on or make changes to this bug.