Skip to content
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-3] Change @font-face and @font-feature-values syntax definitions to use CSS syntax and link to those definitions #624

Closed
SebastianZ opened this issue Oct 20, 2016 · 3 comments
Assignees
Labels
css-fonts-4 Current Work

Comments

@SebastianZ
Copy link
Contributor

SebastianZ commented Oct 20, 2016

The current definition of the <family-name> type is:

The name of a font family of choice such as Helvetica or Verdana in the previous example.

And the <feature-index> and <feature-value-name> types are currently defined like this:

… the @font-feature-values rule is used to define values for a specific font family or set of families that associate a font-specific numeric <feature-index> with a custom <feature-value-name>, which is then used in this property to select specific alternates

This is somewhat vague, as there are no references to the related syntax definitions.

The definitions should be changed to use the syntax from the CSS Syntax Module.

I.e. they may look like this:

@font-face = @font-face { <declaration-list> }

where <declaration-list> can only contain the @font-face descriptors.

and

@font-feature-values = @font-feature-values <family-name># { <rule-list> }

<family-name> = <string> | <custom-ident>+

<feature-value-block> = <feature-type> { <declaration-list> }

<feature-type> = @stylistic | @historical-forms | @styleset | @character-variant | @swash | @ornaments | @annotation

<feature-value> = <integer>+

where <rule-list> can only contain <feature-value-block> rules, <feature-type> takes all font specific property names of font-variant-alternates prefixed with '@', <declaration-list> can only contain feature value definition properties holding <feature-value> values and the <integer> values in <feature-value> denote the feature indices.

The references to these syntax definitions should then link to them.

Sebastian

@tabatkins
Copy link
Member

Updated @font-face and @font-palette-values.

@font-feature-values hasn't been moved to Fonts 4 yet, and I'm not about to go screw around with Fonts 3.

@litherum litherum added css-fonts-3 and removed css-fonts-4 Current Work labels Mar 6, 2018
@svgeesus svgeesus self-assigned this May 1, 2018
@svgeesus
Copy link
Contributor

svgeesus commented May 1, 2018

Yes, I need to port this over to Fonts 4, at which time it will use the CSS Syntax definitions.

We also resolved to remove @font-feature-values from Fonts 3 due to lack of two implementations, so that will deal with the obsolete syntax aspect for that part at least.

@svgeesus
Copy link
Contributor

The relevant section now removed from Fonts 3. I'm editing the text now in Fonts 4 to use the syntax suggested in the initial post.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css-fonts-4 Current Work
Projects
None yet
Development

No branches or pull requests

5 participants