Closed Bug 1737893 Opened 2 years ago Closed 2 years ago

WPT test preserve-3d-flat-grouping-properties.html accidentally squishes its pass/fail output to be unreadable

Categories

(Core :: CSS Parsing and Computation, task)

task

Tracking

()

RESOLVED FIXED
95 Branch
Tracking Status
firefox95 --- fixed

People

(Reporter: dholbert, Assigned: dholbert)

Details

Attachments

(1 file)

WPT test preserve-3d-flat-grouping-properties.html has this styling, which is intended to target all of its tested div elements:

  div {
    width: 200px;
    height: 200px;
  }

Unfortunately, that rule also makes its pass/fail results unreadable (since they are div elements which match that^ CSS rule.)

Let's fix this so that it's easier to see the test-pass/fail results.

Note that this commit shouldn't change the test's pass/fail status anywhere. It
just makes the test results possible to read.

Note, you can see the live test itself here: https://wpt.live/css/css-transforms/preserve-3d-flat-grouping-properties.html

If you scroll down a bit, you can see the squashed pass/fail logging that I'm fixing here.

CC'ing chrishtr who wrote this test -- Chris, feel free to weigh in if you have any concerns about this change. I don't imagine it's controversial, given that it's ~non-functional and just makes the output possible to see. :)

Attachment #9247870 - Attachment description: Bug 1737893: Add wrapper-div to preserve-3d-flat-grouping-properties.html, to avoid accidental restyling of human-readable test-results divs. r?#layout-reviewers → Bug 1737893: Add wrapper-div to WPT test preserve-3d-flat-grouping-properties.html, to avoid accidental restyling of human-readable test-results divs. r?#layout-reviewers
Pushed by dholbert@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/381d9dd110a9
Add wrapper-div to WPT test preserve-3d-flat-grouping-properties.html, to avoid accidental restyling of human-readable test-results divs. r=emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/31389 for changes under testing/web-platform/tests
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch
Upstream PR merged by moz-wptsync-bot
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: