Skip to content

[css-syntax-3] Reference identifier-start instead of name-start#5377

Merged
fantasai merged 2 commits into
w3c:masterfrom
csstools:jn.reference-identifier-start
Jul 31, 2020
Merged

[css-syntax-3] Reference identifier-start instead of name-start#5377
fantasai merged 2 commits into
w3c:masterfrom
csstools:jn.reference-identifier-start

Conversation

@jonathantneal
Copy link
Copy Markdown
Contributor

This updates the label of a reference to the identifier-start code point definition in the CSS Syntax specification.

The identifier code point definition contains a potentially mislabeled reference to a “name-start code point:

identifier code point
A name-start code point, a digit, or U+002D HYPHEN-MINUS (-).

In that definition, “name-start code point” links to the identifier code point definition.

I believe it should be relabeled to “identifier-start code point”:

identifier code point
An identifier-start code point, a digit, or U+002D HYPHEN-MINUS (-).

Comment thread css-syntax-3/Overview.bs
<dd>
A <a>code point</a> with a value equal to or greater than U+0080 &lt;control>.

<dt><dfn export lt="identifier-start code point | name-start code point" oldids="name-start-code-point">identifier-start code point</dfn>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to stay, so old references continue to link.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure thing! This is done in 7103a9f.

Should I also (or alternatively) add anything to oldids?

@jonathantneal jonathantneal requested a review from tabatkins July 31, 2020 22:35
@fantasai fantasai merged commit 3bf2e71 into w3c:master Jul 31, 2020
@jonathantneal jonathantneal deleted the jn.reference-identifier-start branch June 17, 2021 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants