Skip to content

fix: typos across specs - #9164

Merged
tabatkins merged 2 commits into
w3c:mainfrom
nschonni:typos2
Aug 7, 2023
Merged

fix: typos across specs#9164
tabatkins merged 2 commits into
w3c:mainfrom
nschonni:typos2

Conversation

@nschonni

@nschonni nschonni commented Aug 6, 2023

Copy link
Copy Markdown
Contributor

Similar to #5992 this was another pass across the repo with cSpell.
I can rebase out any changes for finalized specs that shouldn't be touched

@tabatkins tabatkins left a comment

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.

A few nits, but r+ once those are fixed.

Comment thread css-color-4/Overview.bs
<wpt>
parsing/color-valid.html
system-color-compute.html
system-color-hightlights-vs-getSelection-001.html

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 is indeed the test name. The typo should probably be fixed in WPT, but it is currently correct.

Comment thread css-color-5/Overview.bs
parsing/color-valid-lab.html
parsing/color-valid-rgb.html
parsing/color-valid-system-color.html
system-color-hightlights-vs-getSelection-001.html

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 test is, again, correctly spelled already.

<th scope="col">%ages
<th scope="col">Anim­ation type
<th scope="col">Canonical order
<th scope="col">Com­puted value

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.

What actually changed in this line?

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.

There was a zero width space or some other odd character that was splitting the word silently

Comment thread css-fonts-4/Overview.bs
<li>Clarified the font shorthand, using the terms set explicitly, reset implicitly and cascade implicitly</li>
<li>Clarified shorter representation for zero ranged descriptors</li>
<li>Tightened definition of first available font</li>
<li>Added emoji as a keyword to unicoderange</li>

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.

What changed here?

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.

The dash was actually a different character code, VSCode was flagging it, so I swapped it for a normal short dash

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.

Okay I thought that might be the case, the standard hyphen-minus is indeed correct.

Comment thread css-nesting-1/proposals.md
<th scope="col">%ages
<th scope="col">Media
<th scope="col">Ani­mat­able
<th scope="col">Com­puted value

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.

What changed in these lines?

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.

Same zero-width character splitting the word

Comment thread css-nesting-1/proposals.md Outdated
@tabatkins
tabatkins merged commit 11b6b73 into w3c:main Aug 7, 2023
@nschonni
nschonni deleted the typos2 branch August 7, 2023 16:01
lu957 added a commit to lu957/csswg-drafts that referenced this pull request Aug 8, 2023
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.

2 participants