Closed Bug 2019254 Opened 2 months ago Closed 1 month ago

Parsing and matching support for multiple comma-separated conditions in @container rules

Categories

(Core :: CSS Parsing and Computation, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
151 Branch
Tracking Status
relnote-firefox --- ?
firefox151 --- fixed

People

(Reporter: jfkthame, Assigned: alaskanemily)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-needed)

Attachments

(2 files)

According to the spec, an @container rule can have multiple comma-separated conditions; we currently only support a single condition. We should extend this to parse and store a list of conditions, and consider the rule to match if any of its conditions matches.

Exposing the list via the CSSContainerRule interface is left for a separate bug (pending spec clarification).

Assignee: nobody → emcdonough
Depends on: 2026769
See Also: → 2022827

[Why is this notable]: New CSS feature
[Affects Firefox for Android]: yes
[Suggested wording]: @container rules now allow to specify a list of container query conditions instead of just one condition.
[Links (documentation, blog post, etc)]:
https://drafts.csswg.org/css-conditional-5/#container-rule
https://developer.mozilla.org/en-US/docs/Web/CSS/Reference/At-rules/@container (describes it but currently still misses a proper example and BCD)

This should probably be mentioned together with the API implemented in bug 2022827.

Sebastian

relnote-firefox: --- → ?
Keywords: dev-doc-needed
Pushed by emcdonough@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/ca67bf8a591b https://hg.mozilla.org/integration/autoland/rev/ab684099bb0e Part 1 - Use indices in CSSContainerRule glue for conditions. r=firefox-style-system-reviewers,emilio https://github.com/mozilla-firefox/firefox/commit/695b0653ab3c https://hg.mozilla.org/integration/autoland/rev/69852bffab54 Part 2 - Implement comma-separated container conditions parsing. r=firefox-style-system-reviewers,emilio
Status: NEW → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 151 Branch
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/58931 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot

Added to the Fx151 nightly release notes, please allow 30 minutes for the site to update.
Keeping the relnote-firefox flag as ? to keep it on the radar for inclusion in the final Fx151 release notes.

QA Whiteboard: [qa-triage-done-c152/b151]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: