- From: Anne van Kesteren <notifications@github.com>
- Date: Tue, 07 Jun 2022 09:20:24 -0700
- To: whatwg/dom <dom@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <whatwg/dom/pull/1079/review/998448149@github.com>
@annevk commented on this pull request. This does not seem equivalent to the sorta-EBNF from before. In particular if the first code point is from BeyondHTMLParserName the second code point was more limited. > -<pre class=lang-ebnf> -<dfn>ValidElementLocalName</dfn> ::= <a>HTMLParserCompatibleName</a> | <a>BeyondHTMLParserName</a> - -<dfn>HTMLParserCompatibleName</dfn> ::= [a-zA-Z] [^#x00#x09#x0A#0x0Cx0D#x20/>]* - -<dfn>BeyondHTMLParserName</dfn> ::= <a>BeyondHTMLParserNameStartChar</a> (<a>BeyondHTMLParserNameChar</a>)* -<dfn>BeyondHTMLParserNameStartChar</dfn> ::= ":" | "_" | [#x80-#x10FFFF] -<dfn>BeyondHTMLParserNameChar</dfn> ::= <a>BeyondHTMLParserNameStartChar</a> | [a-zA-Z] | "-" | "." | [0-9] -</pre> +* its [=string/length=] is at least 1; +* its first [=code point=] is an [=ASCII alpha=], U+003A (:), U+005F (_), or in the range U+0080 to + U+10FFFF inclusive; and comma before inclusive (cannot make code suggestions for some reason) > -<pre class=lang-ebnf> -<dfn>ValidElementLocalName</dfn> ::= <a>HTMLParserCompatibleName</a> | <a>BeyondHTMLParserName</a> - -<dfn>HTMLParserCompatibleName</dfn> ::= [a-zA-Z] [^#x00#x09#x0A#0x0Cx0D#x20/>]* - -<dfn>BeyondHTMLParserName</dfn> ::= <a>BeyondHTMLParserNameStartChar</a> (<a>BeyondHTMLParserNameChar</a>)* -<dfn>BeyondHTMLParserNameStartChar</dfn> ::= ":" | "_" | [#x80-#x10FFFF] -<dfn>BeyondHTMLParserNameChar</dfn> ::= <a>BeyondHTMLParserNameStartChar</a> | [a-zA-Z] | "-" | "." | [0-9] -</pre> +* its [=string/length=] is at least 1; +* its first [=code point=] is an [=ASCII alpha=], U+003A (:), U+005F (_), or in the range U+0080 to + U+10FFFF inclusive; and +* its subsequent [=code points=], if any, are not [=ASCII whitespace=], U+0000 NULL, U+002F (/), or + U+003E (>). Should this be `<ul class=brief>`? I'd prefer we stick with `<p>`s and such for now, though I guess inline square brackets should be fine. -- Reply to this email directly or view it on GitHub: https://github.com/whatwg/dom/pull/1079#pullrequestreview-998448149 You are receiving this because you are subscribed to this thread. Message ID: <whatwg/dom/pull/1079/review/998448149@github.com>
Received on Tuesday, 7 June 2022 16:20:36 UTC