Repository navigation
Add CSS syntax patch for: absolute-size #1794
Description
Link to the definition in the spec
https://drafts.csswg.org/css-fonts-4/#font-size-prop
Syntax of the CSS construct
xx-small | x-small | small | medium | large | x-large | xx-large | xxx-large
Notes
See:
webref/ed/csspatches/syntax-patches.js
Lines 167 to 173 in b338f84
I wasn't sure if the patch is incorrect or if maybe the patch should have been overridden by the later definition from css-fonts-4
Activity
tidoust commentedon Jan 9, 2026
Oh. So the problem is that the
css-fonts-4spec fails to define the<absolute-type>type properly. It defines the construct as avalueinstead. That value links back to the definition of<absolute-type>inCSS2, which is incorrect as that version does not havexxx-large.If there is a proper type definition in
css-fonts-4, it will automatically override the definition inCSS2in Webref. We'll still need a syntax patch, but that patch will then be forcss-fonts-4, and not forCSS2.I'll propose an update to
css-fonts-4. I'd say it's fine to adjust the syntax patch to addxxx-largein the meantime, even if the patch needs to remain tied to the definition inCSS2for the time being.[css-fonts-4] Create proper types for `<absolute-size>` and `<relativ…
[css-fonts-4] Create proper types for `<absolute-size>` and `<relativ…
Drop patches for `<absolute-size>` and `<relative-size>`
Drop patches for `<absolute-size>` and `<relative-size>` (#1796)
tidoust commentedon Jan 12, 2026
Spec was updated. And I was wrong: the syntax in
css-fonts-4can actually be extracted automatically, which means the syntax patch is no longer needed at all. I dropped it.New syntax with
xxx-largewill be released once #1783 can be merged. I'm closing this issue as addressed.Note: The spec encloses the syntax in square brackets, which aren't necessary when considering the syntax in isolation. This is not the only case where enclosing square brackets are used. We could try to normalize syntaxes in the curated data if there's a need for it. Sticking to what the spec defines in the meantime.
romainmenke commentedon Jan 12, 2026
Thank you @tidoust 🙇