-
Notifications
You must be signed in to change notification settings - Fork 637
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
[css-fonts-4] Which type of font family names are system font names? #9292
Comments
Would it be acceptable to define - The following values refer to system fonts: [...]
+ The following values refer to <system-font>` names: [...]
|
I suspect system fonts should have been replaced by installed fonts in e208889, and it should even be entirely removed like in 3d1ac3c. But then I wonder if a system font name should be parsed as a valid
I think
https://drafts.csswg.org/css-fonts-4/#family-name-syntax I suspect it should exclude all pre-defined keywords, regardless of the context in which it appears. |
That clearly means that we need this third type, so it can be explicitly excluded. |
@cdoublev please have a look at 2.1. Font family: the font-family property to let me know if this is headed in the right direction? I think it is clearer now, defining the three types of font family in one place rather than burying one under the |
Looks good, thanks. These clarifications are very helpful.
I think this is new:
But it seems to be matching browser expectations, so I think this is intentional. edit: I mean, |
It is new, because the The older text from which one might infer this is still there:
Although the grammar of the |
Can you please fix this?
(Bikeshed error report) |
Are they generic family names?
https://drafts.csswg.org/css-fonts-4/#font-family-prop
Or a separate type? About
<family-name>#
in the prelude of@font-feature-values
:https://drafts.csswg.org/css-fonts-4/#font-feature-values-syntax
FF currently accepts
@font-feature-values caption {}
whereascaption
is a system font, probably because it wants a<family-name>
that does not match<generic-family>
, which does not explicitly include system font names.aside
font-family: caption
is valid, but it has no effect, therefore it maybe a bit misleading to say this:https://drafts.csswg.org/css-fonts-4/#font-prop
The text was updated successfully, but these errors were encountered: