Closed Bug 1792263 Opened 3 years ago Closed 3 years ago

Fix prefers-contrast media queries in aboutwelcome.scss

Categories

(Firefox :: Messaging System, defect, P2)

defect

Tracking

()

RESOLVED FIXED
107 Branch
Tracking Status
firefox107 --- fixed

People

(Reporter: aminomancer, Assigned: aminomancer)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Firefox generally uses @media (prefers-contrast) for HCM related styles, but we have some rules in aboutwelcome.scss that read @media (prefers-contrast: more). I'm not sure how often this matters, but at least in theory, the HCM styles from common/common-shared.css will be applying to the surfaces that load aboutwelcome.css when prefers-contrast: less or prefers-contrast: custom.

Also, in bug 1790396 Emily added @media (prefers-contrast) rules to _feature-callout.scss, since the rules on Firefox View use that query. So to make sure all these rules are applying under exactly the same conditions, we should use the same query in all cases (unless we end up adding rules for very specific contrast themes, but there don't seem to be any instances of that in the Firefox code, not counting test cases)

I suspect one possible area where this matters may be in forced colors mode, but I haven't done any testing yet.

Priority: -- → P2
Assignee: nobody → shughes
Status: NEW → ASSIGNED
See Also: → 1793401
Pushed by shughes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1a617f6906f0 Consolidate prefers-contrast queries in aboutwelcome. r=pdahiya

The severity field is not set for this bug.
:tspurway, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(tspurway)
Severity: -- → S4
Flags: needinfo?(tspurway)
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 107 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: