Closed Bug 1706366 Opened 4 years ago Closed 4 years ago

The theme icons from the "Themes" slide of the new users Onboarding page are not visible in Windows "High Contrast" mode

Categories

(Firefox :: Messaging System, defect, P2)

Desktop
Windows
defect

Tracking

()

VERIFIED FIXED
90 Branch
Iteration:
90.1 - Apr 19 - May 2
Tracking Status
firefox87 --- unaffected
firefox88 --- unaffected
firefox89 + verified
firefox90 --- verified

People

(Reporter: mcoman, Assigned: Mardak)

References

Details

(Whiteboard: [proton-onboarding] [proton-uplift])

Attachments

(9 files)

Attached image ss of the issue.png (deleted) —

[Affected versions]:

  • Firefox Beta 89.0b1 - Build ID: 20210419195043
  • Firefox Nightly 90.0a1 - Build ID: 20210420095122

[Affected Platforms]:

  • Windows 10 x64

[Prerequisites]:

  • Have a new Firefox profile.
  • Have the "High Contrast" system mode activated.

[Steps to reproduce]:

  1. Open the browser with the profile from the prerequisites.
  2. Click the "Not now" button from the first slide.
  3. Click the "Not now" button from the second slide.
  4. Observe the elements from the "Themes" Onboarding slide.

[Expected result]:

  • All the elements are correctly displayed and respect the "High Contrast" mode.

[Actual result]:

  • The theme icons are not visible.

[Additional Notes]:

  • This issue is not reproducible with the "Themes" modal from the existing/updating users Onboarding.
  • Attached a screenshot of the issue.
Whiteboard: [proton-onboarding]
Priority: -- → P2
Blocks: 1701086

The old behavior uses a background image url while the proton behavior uses a background (image) gradient. Looks like only url() are preserved for high contrast.

Attached image system-theme-swatch.svg (deleted) —
Attached image light-swatch.svg (deleted) —
Attached image dark-swatch.svg (deleted) —
Attached image alpenglow-swatch.svg (deleted) —

Added svg versions of the theme swatches to replace gradient versions.

Assignee: nobody → edilee
Status: NEW → ASSIGNED
Attachment #9219042 - Attachment description: WIP: Bug 1706366 - Change theme icons to SVGs → Bug 1706366 - Update builtin theme icons to reuse on about:welcome, customize and upgrade dialog r?emcminn
Attached image welcome contrast.png (deleted) —
Attached image upgrade contrast.png (deleted) —
Attached image customize chevron down buttons.png (deleted) —
Pushed by elee@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/69811ad1a854 Update builtin theme icons to reuse on about:welcome, customize and upgrade dialog r=emcminn
Flags: needinfo?(edilee)
Pushed by elee@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/68a3d2c973a4 Update builtin theme icons to reuse on about:welcome, customize and upgrade dialog r=emcminn
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
Iteration: --- → 90.1 - Apr 19 - May 2

Comment on attachment 9219042 [details]
Bug 1706366 - Update builtin theme icons to reuse on about:welcome, customize and upgrade dialog r=emcminn a=pascalc

Beta/Release Uplift Approval Request

  • User impact if declined: MR1/Proton about:welcome and upgrade dialog show empty circles in high contrast mode
  • 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: 1. turn on high contrast mode
  1. open about:welcome
  2. move to theme screen and see icons
  • List of other uplifts needed: none
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Primarily updating css to use updated builtin theme icons
  • String changes made/needed: nonee
Flags: needinfo?(edilee)
Attachment #9219042 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Whiteboard: [proton-onboarding] → [proton-onboarding] [proton-uplift]

Comment on attachment 9219042 [details]
Bug 1706366 - Update builtin theme icons to reuse on about:welcome, customize and upgrade dialog r=emcminn a=pascalc

Low risk proton uplift, approved for 89 beta 8, thanks.

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

I have verified that this issue is no longer reproducible with the latest Firefox Nightly (90.0a1 Build ID - 20210504092024) installed on Windows 10 x64. Now I can confirm that the theme icons from the "Themes" slide of the Onboarding page are visible in Windows "High Contrast" mode.

Status: RESOLVED → VERIFIED
Attachment #9219042 - Attachment description: Bug 1706366 - Update builtin theme icons to reuse on about:welcome, customize and upgrade dialog r?emcminn → Bug 1706366 - Update builtin theme icons to reuse on about:welcome, customize and upgrade dialog r=emcminn a=pascalc

Test failure on mozilla-beta but not on mozilla-central was because bug 1702116 landed on m-c and wontfix for beta, so I updated the patch for mozilla-beta copying over 1 line fix.

Depends on: 1702116
Flags: needinfo?(edilee)

Confirming here that we (UX and PM) want this uplifted for MR1 to Beta.

I have verified that this issue is no longer reproducible with the latest Firefox Beta (89.0b9 Build ID - 20210506185706) installed on Windows 10 x64. Now I can confirm that the theme icons from the "Themes" slide of the Onboarding page are visible in Windows "High Contrast" mode.

Flags: qe-verify+
Blocks: 1710864
Regressions: 1712744
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: