Implement CSS parsing support for the @font-palette-values at-rule
Categories
(Core :: CSS Parsing and Computation, enhancement)
Tracking
()
Tracking | Status | |
---|---|---|
firefox107 | --- | fixed |
People
(Reporter: jfkthame, Assigned: jfkthame)
References
Details
Attachments
(4 files)
This is somewhat like the existing @font-feature-settings rule (though simpler), and can be implemented along similar lines.
Assignee | ||
Comment 1•2 years ago
|
||
Not yet hooked up to any rendering functionality.
The intention is for both the @font-palette-values at-rule and the font-palette property
to be behind the same pref being introduced here.
Updated•2 years ago
|
Assignee | ||
Comment 2•2 years ago
|
||
Depends on D157953
Assignee | ||
Comment 3•2 years ago
|
||
The couple of remaining test failures relate to an unclear detail of the spec; see
https://github.com/w3c/csswg-drafts/issues/7308
https://github.com/w3c/csswg-drafts/issues/6916
I've implemented the behavior that I think makes most sense, but the current tests disagree.
Once the CSS WG decides which way to clarify this, we can readily fix either the tests or
our implementation.
Depends on D157954
Assignee | ||
Comment 4•2 years ago
|
||
In https://github.com/w3c/csswg-drafts/issues/6916, the spec text has been updated to clarify
that the font-family descriptor accepts a list of family names.
Depends on D157955
Pushed by jkew@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1d17c257444e patch 1 - Implement CSS parsing for the @font-palette-values rule. r=emilio https://hg.mozilla.org/integration/autoland/rev/7de66a218ba4 patch 2 - Expose @font-palette-values rules in the OM as CSSFontPaletteValuesRule. r=emilio https://hg.mozilla.org/integration/autoland/rev/766386309ee5 patch 3 - Update WPT expectations now that @font-palette-values is supported. r=emilio https://hg.mozilla.org/integration/autoland/rev/a98dd552df64 patch 4 - Adjustments to WPT @font-palette-values parsing tests. r=emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/36327 for changes under testing/web-platform/tests
Comment 7•2 years ago
|
||
Pushed by jkew@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/586c735b005e patch 1 - Implement CSS parsing for the @font-palette-values rule. r=emilio https://hg.mozilla.org/integration/autoland/rev/cf87497dbead patch 2 - Expose @font-palette-values rules in the OM as CSSFontPaletteValuesRule. r=emilio https://hg.mozilla.org/integration/autoland/rev/5e32d5a01917 patch 3 - Update WPT expectations now that @font-palette-values is supported. r=emilio https://hg.mozilla.org/integration/autoland/rev/e317795cd27a patch 4 - Adjustments to WPT @font-palette-values parsing tests. r=emilio
Comment 9•2 years ago
|
||
Backed out for causing build bustages on TestCOLRv1.cpp
- Backout link
- Push with failures
- Failure Log
- Failure line: /builds/worker/checkouts/gecko/gfx/tests/fuzz/TestCOLRv1.cpp:57:55: error: no viable conversion from 'mozilla::gfx::sRGBColor' to 'const mozilla::gfx::Point' (aka 'const PointTyped<mozilla::gfx::UnknownUnits>')
Comment 10•2 years ago
|
||
Pushed by jkew@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/24ce615e77fd patch 1 - Implement CSS parsing for the @font-palette-values rule. r=emilio https://hg.mozilla.org/integration/autoland/rev/a47ae5f76514 patch 2 - Expose @font-palette-values rules in the OM as CSSFontPaletteValuesRule. r=emilio https://hg.mozilla.org/integration/autoland/rev/af4243189ef1 patch 3 - Update WPT expectations now that @font-palette-values is supported. r=emilio https://hg.mozilla.org/integration/autoland/rev/6585dd386ee3 patch 4 - Adjustments to WPT @font-palette-values parsing tests. r=emilio
Comment 11•2 years ago
|
||
Backed out 9 changesets (Bug 1791778, Bug 1791782, Bug 1791780, Bug 1791777) for causing build bustages in TestCOLRv1.cpp CLOSED TREE
Log: https://treeherder.mozilla.org/logviewer?job_id=392630966&repo=autoland&lineNumber=15311
Backout: https://hg.mozilla.org/integration/autoland/rev/f1accadf1754ad5aa8d668ecb22ec8ef1a5bd880
Comment 12•2 years ago
|
||
Pushed by jkew@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9d7de4f1be55 patch 1 - Implement CSS parsing for the @font-palette-values rule. r=emilio https://hg.mozilla.org/integration/autoland/rev/e1b9106de285 patch 2 - Expose @font-palette-values rules in the OM as CSSFontPaletteValuesRule. r=emilio https://hg.mozilla.org/integration/autoland/rev/eb4554c39a2f patch 3 - Update WPT expectations now that @font-palette-values is supported. r=emilio https://hg.mozilla.org/integration/autoland/rev/a588e88a1d8b patch 4 - Adjustments to WPT @font-palette-values parsing tests. r=emilio
Comment 13•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9d7de4f1be55
https://hg.mozilla.org/mozilla-central/rev/e1b9106de285
https://hg.mozilla.org/mozilla-central/rev/eb4554c39a2f
https://hg.mozilla.org/mozilla-central/rev/a588e88a1d8b
Assignee | ||
Updated•2 years ago
|
Upstream PR merged by moz-wptsync-bot
Description
•