Closed Bug 1835066 Opened 7 months ago Closed 6 months ago

[css-nesting] Enable on Nightly

Categories

(Core :: CSS Parsing and Computation, task)

task

Tracking

()

RESOLVED FIXED
115 Branch
Tracking Status
firefox115 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

(Keywords: dev-doc-complete)

Attachments

(5 files)

No description provided.

Even though there's some implementation work left before it being in a
shippable state, the bits that do work (which is quite a few) should
work reliably and performantly.

Given this is (afaict) the first implementation of the new spec, I want
to let people test it easier, specially since there are some questions
about error recovery that nesting could change.

Without nesting, the parser error is the unexpected token error, emitted
from cssparser.

With nesting enabled these behave the same as other strings outside of a
block (so an "unexpected end of input" error instead).

Tweak the test a bit to keep testing what this test wants to test (that
unexpected token errors are escaped).

Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7cae8c946ae5
Remove now-valid selector from file_bug416317.xhtml. r=dholbert
Keywords: leave-open
Attachment #9336994 - Attachment description: WIP: Bug 1835066 - Make our nesting implementation match the new spec better. → Bug 1835066 - Make our nesting implementation match the new spec better. r=tlouw,#style,#layout
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7c8f755bb185
Tweak test_parser_diagnostics_unprintables.html to keep testing unexpected token errors. r=dholbert
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/af7ff8bcf986
Remove some WPTs which aren't valid in presence of nesting. r=dholbert
https://hg.mozilla.org/integration/autoland/rev/6d74219ed171
Make our nesting implementation match the new spec better. r=tlouw
https://hg.mozilla.org/integration/autoland/rev/96604e3a217e
[css-nesting] Enable on nightly. r=dholbert
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/40376 for changes under testing/web-platform/tests
Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/autoland/rev/00802389f336
Fix a Gecko reftest that isn't valid when nesting is enabled.
Regressions: 1836670
Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/autoland/rev/81afd61afe48
Disable a test on opt builds for now.
Keywords: leave-open
Regressions: 1836745
Upstream PR merged by moz-wptsync-bot

A number of pages have been created:

  • /en-US/docs/Web/CSS/CSS_nesting
  • /en-US/docs/Web/CSS/Nesting_selector
  • /en-US/docs/Web/CSS/CSS_nesting/Using_CSS_nesting
  • /en-US/docs/Web/CSS/CSS_nesting/Nesting_and_specificity
  • /en-US/docs/Web/CSS/CSS_nesting/Nesting_at-rules

Added to the release notes for Firefox 117:

  • /en-US/docs/Mozilla/Firefox/Releases/117

Updated the CSSRef.ejs page in Yari to show the new pages in Guides

Associated Pull Requests

You need to log in before you can comment on or make changes to this bug.