Skip to content

[css-fonts-4] Group sub-property related symbols in font-variant value definition #7683

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

Closed
cdoublev opened this issue Sep 3, 2022 · 1 comment · Fixed by #7742
Closed
Labels
Closed Accepted as Editorial Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-fonts-4 Current Work

Comments

@cdoublev
Copy link
Collaborator

cdoublev commented Sep 3, 2022

If the following change is not against any non-defined rules applying to the CSS value definition syntax, I would like each font-variant sub-properties to have its corresponding symbols grouped (between []) in the font-variant value definition.

A CSS parser could automatically output a nested list of the component values matching the grouped symbols. This would avoid coupling the expansion function of font-variant to its current value definition.

    normal 
  | none 
  | [
-        <common-lig-values> || <discretionary-lig-values> || <historical-lig-values> || <contextual-alt-values> 
+        [<common-lig-values> || <discretionary-lig-values> || <historical-lig-values> || <contextual-alt-values>]
      || [small-caps | all-small-caps | petite-caps | all-petite-caps | unicase | titling-caps] 
      || [
              stylistic(<feature-value-name>) 
           || historical-forms 
           || styleset(<feature-value-name>#) 
           || character-variant (<feature-value-name>#) 
           || swash(<feature-value-name>) 
           || ornaments(<feature-value-name>) 
           || annotation(<feature-value-name>)
         ] 
-     || <numeric-figure-values> || <numeric-spacing-values> || <numeric-fraction-values> || ordinal || slashed-zero 
+     || [<numeric-figure-values> || <numeric-spacing-values> || <numeric-fraction-values> || ordinal || slashed-zero]
-     || <east-asian-variant-values> || <east-asian-width-values> || ruby 
+     || [<east-asian-variant-values> || <east-asian-width-values> || ruby]
      || [sub | super] 
      || [text | emoji | unicode] 
   ]
@svgeesus
Copy link
Contributor

That seems reasonable to me

@svgeesus svgeesus added css-fonts-4 Current Work Closed Accepted as Editorial Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. labels Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Accepted as Editorial Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-fonts-4 Current Work
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants