Closed Bug 1700728 Opened 4 years ago Closed 4 years ago

Firefox-only WPT failures in flex-computed.html and flex-flow-computed.html

Categories

(Core :: Layout: Flexbox, defect, P3)

Product:
Core
Component:
Layout: Flexbox
Type:
defect

Tracking

()

Status:
VERIFIED DUPLICATE of bug 137688

People

(Reporter: dholbert, Unassigned)

References

(Blocks 1 open bug)

Details

I do see this error reported in my devtools console when loading the page:

Dangling combinator.  Ruleset ignored due to bad selector.

I'm not sure what combinator that's referring to, but it does sound like the sort of thing that could cause all tests to fail...

Ah, no -- this is actually bug 137688. The test is asking to read back the computed style of a shorthand property ("flex").

So, this is a version of bug 137688.

Depends on: 137688

I'll extend this to cover flex-flow.html as well, since it looks like it's the same issue, with that shorthand instead of flex.

Handy links for that one:
https://wpt.fyi/results/css/css-flexbox/parsing/flex-flow-computed.html?label=master&label=experimental&aligned
https://wpt.live/css/css-flexbox/parsing/flex-flow-computed.html
https://searchfox.org/mozilla-central/rev/368607c4cd5be547021945e4ae60e8eb4365b3c4/testing/web-platform/meta/css/css-flexbox/parsing/flex-flow-computed.html.ini

Summary: Firefox-only WPT failures in flex-computed.html → Firefox-only WPT failures in flex-computed.html and flex-flow-computed.html
Blocks: compat2021

Do you want to keep this open? We should just fix bug 137688 more generally.

Severity: -- → S3
Priority: -- → P3

Yeah, might as well just dupe to bug 137688 for now. We can reopen if there ends up being anything more to fix for these tests.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → DUPLICATE

These WPT tests pass now. \o/

Thanks for fixing bug 137688, emilio!

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.