-
Notifications
You must be signed in to change notification settings - Fork 189
Update WCAG references from 2.0 to 2.2 #5681
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
base: main
Are you sure you want to change the base?
Conversation
Fixes canonical#5411 Updated all outdated WCAG 2.0 references to WCAG 2.2 in the accessibility documentation: - Updated text from "WCAG 2.0 website" to "WCAG 2.2 website" - Updated WCAG links from 2.0 to 2.2 versions - Updated link labels from "WCAG 2.0" to "WCAG 2.2" All links now point to the current WCAG 2.2 standards and resources.
|
Nayab-Gauhar is not a collaborator of the repo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR reformats the accessibility guidelines HTML template for improved code consistency and updates WCAG references from version 2.0 to 2.2.
- Standardizes indentation throughout the template
- Updates all WCAG 2.0 references to WCAG 2.2 with corresponding URL updates
- Removes unnecessary line breaks (
<br>tags) and cleans up whitespace
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jmuzina
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WCAG reference updates themselves look good - some minor comments about formatting and possibly out of scope changes below.
| <div class="p-strip"> | ||
| <div class="u-fixed-width"> | ||
| <h2>Noticed an issue?</h2> | ||
| <p>If you spot an accessibility problem in Vanilla, let us know <br> by <a href="https://github.com/canonical/vanilla-framework/issues">filing an issue</a> on GitHub.</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why remove the nbsp's here?
They are helping to keep the link as a whole on one line as the page shrinks. This is a very minor change, as this link only wraps on exceptionally small screens, but I'd like to keep the changes here in scope to the WCAG reference updates.
| <div class="row"> | ||
| <div class="col-12"> | ||
| <h1>Accessibility guidelines</h1> | ||
| <p>Vanilla Framework aims for Level AA conformance with the <br><a href="https://www.w3.org/TR/WCAG22/">Web Content Accessibility Guidelines (WCAG) 2.2</a></p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please keep the BR, it helps ensure WCAG starts on a new line
Fixes #5411
Summary
Updated all outdated WCAG 2.0 references to WCAG 2.2 in the accessibility documentation page.
Changes
https://www.w3.org/TR/WCAG20/→https://www.w3.org/TR/WCAG22/https://www.w3.org/TR/UNDERSTANDING-WCAG20/→https://www.w3.org/WAI/WCAG22/Understanding/https://www.w3.org/WAI/WCAG20/quickref/→https://www.w3.org/WAI/WCAG22/quickref/https://www.w3.org/TR/WCAG20-TECHS/→https://www.w3.org/WAI/WCAG22/Techniques/All links now point to the current WCAG 2.2 standards and resources.