1783780 - Update image icon used for default theme in MR onboarding colorway screen

Status: VERIFIED FIXED

(Firefox :: Messaging System, enhancement, P1)

Description

[Punam Dahiya [:pdahiya]]

Scope of this bug is to update icon used for default theme

Comment 1

[Punam Dahiya [:pdahiya]]

Ni Ania to help with icon used in default theme svg on colorway screen

Comment 2

[Ania]

Attachment: Default.svg

Hi Punam,
Apologies for the delay with this, here's the icon.

Comment 3

[Punam Dahiya [:pdahiya]]

Attachment: Bug 1783780 - Update image icon used for default theme in MR onboarding colorway screen

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/42ba742c1218
Update image icon used for default theme in MR onboarding colorway screen r=mviar

Comment 5

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/42ba742c1218

Comment 6

[Simona Rosu [:srosu], Ecosystem QA]

Attachment: currentImplementationColors.png

I‘ve verified this enhancement using the latest Firefox Nightly 105.0a1 (Build ID: 20220818214910) on Windows 10 x64, macOS 12.3.1, and Linux Mint 20.2 x64.

• The default theme icon is updated with the attached asset.

However, during verification, I observed that the colorway icons from Figma have a more pronounced color than in the current implementation. @Punam, is this difference expected?

Comment 7

[Punam Dahiya [:pdahiya]]

Thanks @srosu that's expected as colorways closet uses icon available with respective colorways e.g. resource://builtin-themes/colorways/2022playmaker/balanced/icon.svg . Left comment in figma for UX to confirm that's expected behavior

https://www.figma.com/file/2TrMkNefxzhMCXjUqM8KIy?node-id=4854:116803#252469886

Comment 8

[Simona Rosu [:srosu], Ecosystem QA]

Thanks, Punam! I'm marking this enhancement as Verified based on comments 6 and 7.

Comment 9

[Gabrielle Lussier [:glussier]]

Thanks for flagging this Punam! Yep, I'm expecting it to appear as it does in the Colorways modal. I stole these designs from Victoria's figma file but they may be outdated/not represent the actual implementation.