Closed Bug 1789712 Opened 2 years ago Closed 2 years ago

HCM: background images are not allowed in property with multiple backgrounds if one of the layers lacks a background image

Categories

(Core :: Layout, defect)

defect

Tracking

()

VERIFIED FIXED
106 Branch
Accessibility Severity s3
Tracking Status
firefox106 --- verified

People

(Reporter: aminomancer, Assigned: emilio)

References

(Blocks 1 open bug, )

Details

(Keywords: access, testcase)

Attachments

(1 file)

(Tested on Windows 10 and macOS 12.5.1)

STR

  1. Make sure HCM is disabled
  2. Open this test case
  3. Click the lightning bolt a few times, see that the background color alternates between red and blue and the bolt remains visible
  4. Now set browser.display.document_color_use to 2
  5. Go back to the test case - the red/blue background should no longer be visible, overridden by HCM
  6. Again, click the lightning bolt a few times

Expected

The bolt should remain visible

Actual

In HCM, the bolt disappears for the blue background mode. Looking at the changeBG function in the JS section, you can see the problem. It's replacing red with , blue which means an additional background layer. Since this layer has no image, the background-image in the first layer isn't honored, so we see no bolt in this state.

This affects the noodle images on the new MR about:welcome (browser.aboutwelcome.templateMR = true). I submitted a provisional patch for that issue (bug 1789667) which just consolidates the background-color and background-image into one layer, but it would still be worth fixing the underlying cause. I haven't done an exhaustive review of what else this may affect.

Blocks: hcm
Flags: needinfo?(emilio)
Assignee: nobody → emilio
Status: NEW → ASSIGNED
Flags: needinfo?(emilio)
Keywords: access
Whiteboard: [access-s3]
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e466469ec2c4 Allow backplating if there's a background-image which is `none`. r=morgan

Nice! Thanks

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 106 Branch
Flags: qe-verify+

Reproduced on build from 06.09.2022.
No longer reproducible on 106.0b4

Status: RESOLVED → VERIFIED
Flags: qe-verify+
Accessibility Severity: --- → s3
Whiteboard: [access-s3]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: