Closed Bug 1801037 Opened 2 years ago Closed 2 years ago

Remove "New colorways are on the way" message from Firefox View

Categories

(Firefox :: Firefox View, task, P1)

task
Points:
5

Tracking

()

VERIFIED FIXED
109 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox107 --- wontfix
firefox108 --- verified
firefox109 --- verified

People

(Reporter: dao, Assigned: dao)

References

Details

(Whiteboard: [fidefe-firefox-view])

Attachments

(2 files)

As we don't have plans for a followup collection, we need to remove the "New colorways are on the way" message.

Whiteboard: [fidefe-firefox-view]
Points: --- → 5
Attachment #9303815 - Attachment description: Bug 1801037 - Remove "New colorways are on the way" message from Firefox View. r=gijs → Bug 1801037 - Remove "New colorways are on the way" message from Firefox View. r=sfoster
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/dfe42e743b9c Remove "New colorways are on the way" message from Firefox View. r=sfoster
Attachment #9303818 - Attachment description: Bug 1801037 - Disable "New colorways are on the way" message (patch for beta uplift). r=gijs → Bug 1801037 - Disable "New colorways are on the way" message (patch for beta uplift). r=sfoster
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch

Comment on attachment 9303818 [details]
Bug 1801037 - Disable "New colorways are on the way" message (patch for beta uplift). r=sfoster

Beta/Release Uplift Approval Request

  • User impact if declined: Misleading message as we don't have plans for another colorway collection
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce:
    • set your system date to something after January 17, 2023
    • launch Firefox
    • open Firefox View
      -> the feature callout tour should skip the colorways step
      -> the page should not have a colorway card, while the page layout should otherwise be unaffected from the card being hidden
    • resize the window a few times and confirm that the page layout still works out
  • List of other uplifts needed: bug 1801044
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This is a simplified patch specifically for beta to reduce risk. (Technically the card is still there, but it's unstyled and hidden.)
  • String changes made/needed: n/a
  • Is Android affected?: No
Attachment #9303818 - Flags: approval-mozilla-beta?
Flags: qe-verify+

I thought we were targeting this for 109, not 108?

(In reply to Sarah Clements [:sclements] from comment #6)

I thought we were targeting this for 109, not 108?

Many users (possibly even most users) may still be on 108 as the collection expires on Jan 17, the 109 release day.

Comment on attachment 9303818 [details]
Bug 1801037 - Disable "New colorways are on the way" message (patch for beta uplift). r=sfoster

Approved for 108.0b4

Attachment #9303818 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
QA Whiteboard: [qa-triaged]

Verified the STR from Comment 5 with Firefox 108.0b4 and 109.0a1 (2022-11-20) on macOS 13.0.1, Ubuntu 22.04 and Windows 11. The colorway step is skipped, and the colorway card is hidden as expected.

The only visible thing would be that the content in page without the colorway card now looks a bit shifted to the left and not centered, otherwise no other issues were found. Let me know if I can help with anything else.

Status: RESOLVED → VERIFIED
Flags: qe-verify+

(In reply to Catalin Sasca, QA [:csasca] from comment #10)

The only visible thing would be that the content in page without the colorway card now looks a bit shifted to the left and not centered, otherwise no other issues were found.

Afaik this is expected. Thanks!

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: