Skip to content

"Web" to "web", "Success criteria" to "success criteria" #4428

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

Open
wants to merge 30 commits into
base: main
Choose a base branch
from

Conversation

patrickhlauke
Copy link
Member

@patrickhlauke patrickhlauke commented Jun 4, 2025

Also includes:

  • change "web based" to "web-based"
  • change "Web Unit" to "web unit"
  • change "Web Page" to "web page" (where appropriate)
  • a few straggler cases where lowercase "success criterion/criteria" was referring to specific ones, where we do want title case

Closes #4427

EDIT: see the matching #4461 that takes care of normative documents

…s criteria"

Also includes:

* change "web based" to "web-based"
* change "Web Unit" to "web unit"
* change "Web Page" to "web page" (where appropriate)

Closes #4427
Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 41e4405
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/6840b7564d258000080d37cc
😎 Deploy Preview https://deploy-preview-4428--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit f9f0cd6
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/6840b7a720af8c0008d74bfa
😎 Deploy Preview https://deploy-preview-4428--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 161fd94
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/6840b808525d1600081b591f
😎 Deploy Preview https://deploy-preview-4428--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 45875d6
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/68713db2bb97fc00084c854d
😎 Deploy Preview https://deploy-preview-4428--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@patrickhlauke patrickhlauke changed the title Change "Web" to "web", two instances of "Success criteria" to "success criteria" Editorial: "Web" to "web", "Success criteria" to "success criteria" Jun 4, 2025
@kfranqueiro
Copy link
Contributor

We should skip the HTML files under conformance-challenges and requirements

@gundulaniemann
Copy link
Contributor

I learned that terms like "section" and "success criterion" are supposed to be written starting with a capital letter if a number follows, like "Section 4" or "Success Criterion 1.4.11".
Is that not true? (or has changed over the past decades?)

@patrickhlauke
Copy link
Member Author

I learned that terms like "section" and "success criterion" are supposed to be written starting with a capital letter if a number follows, like "Section 4" or "Success Criterion 1.4.11". Is that not true? (or has changed over the past decades?)

and are any of the changes here contradicting that?

actually, though i wasn't specifically looking for that case, it seems there's some cases in the current wording of certain files where it's right now lowercase when it shouldn't be, so will actually tackle that now here too

@patrickhlauke
Copy link
Member Author

thanks for checking @kfranqueiro ... I'm glad, as the whole "the Web..." usage feels archaic (and always makes me think of "the Grid" https://www.youtube.com/watch?v=lILHEnz8fTk)

@patrickhlauke
Copy link
Member Author

patrickhlauke commented Jun 16, 2025

so, unless there's any more quirks/corrections ... i'm happy to split this into two (normative and non-normative) for the next meeting

edit: @kfranqueiro also "happy" to report that your new PR check for normative/non-normative mixed updates kicked in as it should https://github.com/w3c/wcag/actions/runs/15681204166/job/44172828352?pr=4428

@patrickhlauke patrickhlauke self-assigned this Jun 16, 2025
@bruce-usab bruce-usab self-assigned this Jun 17, 2025
@bruce-usab
Copy link
Contributor

It could be a separate PR, but "web unit" should be deleted or replaced with "web page". The term was a placeholder while AG wrestled with a putative last call working draft, see 2006 glossary.

Per @bruce-usab's comment, removed one instance altogether as it was unnecessary, and replaced another with "set of web pages" which is the correct term here that matches the wording of the related SC
@patrickhlauke
Copy link
Member Author

@bruce-usab killed off those two straggler "web unit"s

Any more? Otherwise, will proceed to disentangle and make two separate PRs out of this

@bruce-usab
Copy link
Contributor

Discussed on backlog call 6/20, and @patrickhlauke ready to close this out.

@patrickhlauke
Copy link
Member Author

@kfranqueiro I've split this out now, so the PR here is only non-normative. Normative changes are in #4461

@patrickhlauke patrickhlauke changed the title Editorial: "Web" to "web", "Success criteria" to "success criteria" Editorial (Non-normative): "Web" to "web", "Success criteria" to "success criteria" Jun 22, 2025
@mbgower mbgower changed the title Editorial (Non-normative): "Web" to "web", "Success criteria" to "success criteria" "Web" to "web", "Success criteria" to "success criteria" Jun 24, 2025
@@ -133,7 +133,7 @@ <h2>Examples of Images of Text</h2>
<h2>Resources for Images of Text</h2>
<ul>
<li><a href="https://developer.mozilla.org/en-US/docs/Learn_web_development/Core/Text_styling/Fundamentals">Fundamental text and font styling</a></li>
<li><a href="https://developer.mozilla.org/en-US/docs/Learn_web_development/Core/Text_styling/Web_fonts">MDN Web fonts</a></li>
<li><a href="https://developer.mozilla.org/en-US/docs/Learn_web_development/Core/Text_styling/Web_fonts">MDN: Web fonts</a></li>
Copy link
Contributor

Choose a reason for hiding this comment

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

Line 126 in previous file (images of text no exception) uses web (lowercase) fonts

Copy link
Member Author

@patrickhlauke patrickhlauke Jul 7, 2025

Choose a reason for hiding this comment

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

decide which one we should use. The title of the page itself is "Web fonts". Prefixing it with "MDN:" it probably makes sense to keep the uppercase "W" (so we need to change it in the no exception variant?)

Copy link
Contributor

@mbgower mbgower Jul 11, 2025

Choose a reason for hiding this comment

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

I'm inclined to leave this as is. It is the title of the page. I don't think we inflict our style on referenced material.
Update: the page is titled "Web fonts". The "MDN" has been prefixed for context. It doesn't look to me like we normally cite the source in our reference links, so my inclination is to removed "MDN"

@@ -123,7 +123,7 @@ <h2>Resources for Images of Text (No Exception)</h2>

<li>

<a href="http://alistapart.com/article/cssatten">CSS Web fonts</a>
<a href="http://alistapart.com/article/cssatten">CSS web fonts</a>
Copy link
Contributor

Choose a reason for hiding this comment

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

Line 136 next in file list (images of text) uses Web (uppercase) fonts.

Copy link
Member Author

Choose a reason for hiding this comment

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

ah, this is the "other" file. In this case makes sense to keep lowercase.

Copy link
Contributor

@mbgower mbgower Jul 11, 2025

Choose a reason for hiding this comment

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

isn't the title of this article "CSS @ Ten: The Next Big Thing"? Do we normally make up our own link names, or use the title of the source?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Some missed editorial changes from the Dec 2024 WCAG publication
5 participants