Re: [csswg-drafts] [css-fonts] Drop bracket matching step from `@font-face` src: line parsing (#6340)

Thanks for commenting, @litherum. To summarize, I suggest to improve this section by dropping 

> The value of the 'src!!descriptor' descriptor is parsed piecewise. First, opening and closing parentheses are matched. If these parentheses cannot be matched, the value is a parse error. Matching the parentheses has the effect of partitioning the value into multiple regions, each of which is either inside outermost parentheses or outside outermost parentheses. 

> Once the value is partitioned, all the commas outside outermost parentheses are located. The value is then split at these comma locations.

> Each item in the split value is then parsed against this grammar:

and instead say:

> The src: descriptor value must be parsed according to section ["5.3.10. Parse a comma-separated list of component values"](https://drafts.csswg.org/css-syntax/#parse-comma-separated-list-of-component-values). Then each component value is parsed according to this grammar: <grammar>

I also suggest to address "ISSUE 4" and the definition of "thrown out" by saying:

> If a component value is parsed correctly and is of a format and font technology that the UA supports, add it to the list of supported sources. If parsing a component value results in a parsing error or its format or technology are unsupported, do not add it to the list of supported sources. 

> If there are no supported entries at the end of this process, the value for the src: descriptor is a parse error. 






-- 
GitHub Notification of comment by drott
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/6340#issuecomment-857708634 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Wednesday, 9 June 2021 13:47:43 UTC