Skip to content
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

Replaces the previous name of the working group in Understanding #4257

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

Conversation

momdo
Copy link
Contributor

@momdo momdo commented Mar 1, 2025

Replace "WCAG" and "Web Content Accessibility Guidelines" with "Accessibility Guidelines".

Copy link

netlify bot commented Mar 1, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 24dee34
🔍 Latest deploy log https://app.netlify.com/sites/wcag2/deploys/67cc1a4c3ef988000856ff2c
😎 Deploy Preview https://deploy-preview-4257--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 site configuration.

@patrickhlauke
Copy link
Member

This one is likely a bit more controversial/not as straightforward, as at the time that WCAG came out, I think the WG was still called that. I think it's possibly safer to leave as is, or to neutralise it by just referring to "the Working Group" to make it applicable to both the old and new names.

@patrickhlauke patrickhlauke self-assigned this Mar 1, 2025
@mbgower mbgower self-requested a review March 7, 2025 16:45
@mbgower
Copy link
Contributor

mbgower commented Mar 7, 2025

A search on "WCAG working group" revealed 36 results over 19 files in the repo -- more than the 6 files changed here.
I'm going to review each of the other occurrences to confirm if they should likewise be altered.

Note that I also confirmed that many occurrences of "Accessibility Guidelines Working Group" exist, so the proposed term is certainly used in the index file and elsewhere.

@alastc note that I did discover inconsistency between the use of "AG WG" (53 results in 20 files) and "AGWG" (5 results in 3 files) in usage. I am going to correct the far less prevalent occurrences of AGWG at the same time.

@mbgower
Copy link
Contributor

mbgower commented Mar 7, 2025

I've determined that other occurrences seem to occur either in old wcag 2 areas not to be updated or in the specification, not relevant non-normative documents, so I will create a separate PR for those the spec changes.

IMO this change looks solid.

@mbgower
Copy link
Contributor

mbgower commented Mar 7, 2025

@momdo I cannot find your PR, so I could not make additional changes in it.
One thing I noticed: you missed a few other corrections where the case was not correct. Would you please do a search for
"WCAG Working group" and "WCAG working group" and ensure both are changed to "AG Working Group"

@kfranqueiro this ID is not in the group. What's the easiest way to address?

@bruce-usab
Copy link
Contributor

Discussed during Backlog call today and Mike is on the case!

@momdo
Copy link
Contributor Author

momdo commented Mar 8, 2025

@mbgower
I'm sorry for the inconvenience...
I found "WCAG Working group" in understanding/conformance.html, so I replaced it.
Could you please check if this commit is correct?

@@ -642,7 +642,7 @@ <h3>Level of Assistive Technology Support Needed for "Accessibility Support"</h3

<p>This topic raises the question of how many or which assistive technologies must support
a Web technology in order for that Web technology to be considered "accessibility
supported". The WCAG Working group and the W3C do not specify which or how many assistive
supported". The AG Working Group and the W3C do not specify which or how many assistive
Copy link
Contributor

@kfranqueiro kfranqueiro Mar 10, 2025

Choose a reason for hiding this comment

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

Would we rather expand this to Accessibility Guidelines in all cases? (Other instances have been expanded from WCAG to Accessibility Guidelines in this PR, but this one is still abbreviated as AG. I realize this might be due to the phrasing in @mbgower's comment...)

Copy link
Member

Choose a reason for hiding this comment

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

I'd be in favour of expanding in all instances, yes

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
supported". The AG Working Group and the W3C do not specify which or how many assistive
supported". The Accessibility Guidelines Working Group and the W3C do not specify which or how many assistive

@w3cbot
Copy link

w3cbot commented Mar 10, 2025

kfranqueiro marked as non substantive for IPR from ash-nazg.

@kfranqueiro
Copy link
Contributor

@kfranqueiro this ID is not in the group. What's the easiest way to address?

I've marked this as non-substantive (which I think is fair for this PR?) in order to pass the check.

@patrickhlauke patrickhlauke self-requested a review March 10, 2025 11:29
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.

6 participants