-
Notifications
You must be signed in to change notification settings - Fork 718
[cssom][css-conditional] CSSGroupingRule interface is duplicated #3528
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Just stumbled over this myself. As @foolip mentioned, there are a few discrepancies between the two. In particular, CSSOM defines an This seems to be a source of confusion since Chromium requires the argument, but Firefox and WebKit do not:
For what it's worth, an optional argument is more consistent with the |
CSSGroupingRule is defined by CSSOM.
These tests are somewhat contentious due an inconsistency in the CSS specification: w3c/csswg-drafts#3528
These tests are somewhat contentious due an inconsistency in the CSS specification: w3c/csswg-drafts#3528
These tests are somewhat contentious due an inconsistency in the CSS specification: w3c/csswg-drafts#3528
We resolved this issue via gh-4029 though I don't have the permissions to close it. |
These tests are somewhat contentious due an inconsistency in the CSS specification: w3c/csswg-drafts#3528
…nsertRule`, a=testonly Automatic update from web-platform-tests [cssom] Add tests for CSSGroupingRule These tests are somewhat contentious due an inconsistency in the CSS specification: w3c/csswg-drafts#3528 -- wpt-commits: 814db842c492f8c480749b671e6bc31c83c68645 wpt-pr: 17276
…nsertRule`, a=testonly Automatic update from web-platform-tests [cssom] Add tests for CSSGroupingRule These tests are somewhat contentious due an inconsistency in the CSS specification: w3c/csswg-drafts#3528 -- wpt-commits: 814db842c492f8c480749b671e6bc31c83c68645 wpt-pr: 17276
These tests are somewhat contentious due an inconsistency in the CSS specification: w3c/csswg-drafts#3528
…nsertRule`, a=testonly Automatic update from web-platform-tests [cssom] Add tests for CSSGroupingRule These tests are somewhat contentious due an inconsistency in the CSS specification: w3c/csswg-drafts#3528 -- wpt-commits: 814db842c492f8c480749b671e6bc31c83c68645 wpt-pr: 17276 UltraBlame original commit: 6095a82d809c716895d2e1e2075cb59e3072bc42
…nsertRule`, a=testonly Automatic update from web-platform-tests [cssom] Add tests for CSSGroupingRule These tests are somewhat contentious due an inconsistency in the CSS specification: w3c/csswg-drafts#3528 -- wpt-commits: 814db842c492f8c480749b671e6bc31c83c68645 wpt-pr: 17276 UltraBlame original commit: 6095a82d809c716895d2e1e2075cb59e3072bc42
…nsertRule`, a=testonly Automatic update from web-platform-tests [cssom] Add tests for CSSGroupingRule These tests are somewhat contentious due an inconsistency in the CSS specification: w3c/csswg-drafts#3528 -- wpt-commits: 814db842c492f8c480749b671e6bc31c83c68645 wpt-pr: 17276 UltraBlame original commit: 6095a82d809c716895d2e1e2075cb59e3072bc42
The CSSGroupingRule interface is defined in two specs:
https://drafts.csswg.org/cssom/#the-cssgroupingrule-interface
https://drafts.csswg.org/css-conditional/#the-cssgroupingrule-interface
They are not identical, but the one in CSSOM looks like the one to keep.
The text was updated successfully, but these errors were encountered: