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

in brief material for WCAG 2.0 success criteria #3219

Merged
merged 31 commits into from Aug 1, 2023
Merged

Conversation

mbgower
Copy link
Contributor

@mbgower mbgower commented May 29, 2023

Brief descriptions for all WCAG 2.0 A and AA requirements. A continuation of the work begun in #3191 and #2905 to address #744

I set as a goal to limit each line to 10 words or fewer
For the benefits, I tried to reuse the same phrases where possible. I've documented the phrases used and some explanation in the following document
Key beneficiaries language.docx

Brief descriptions for all WCAG 2.0 A and AA requirements
@gundulaniemann
Copy link

gundulaniemann commented Jun 9, 2023

I understand that larger feedback should be given in github instead of in the questionnaire.
If there is a better way than a comment, I am happy to learn.

Here are my remarks on the current version:

Understanding Contrast (Minimum)
there are various vision impairments where users rely on good contrast.
what about:

Key beneficiaries
Users with vision impairments
I'd also agree to "Users with reduced vision"

Understanding Error Prevention (Legal, Financial, Data)
I do not see the impact on users with reduced hearing.
Can you explain?

Understanding Info and Relationships
As the information should be programmatically determined, I suggest to add blind users to the list of beneficiaries.

Understanding Keyboard
What about a positive wording like
"Enable accomplishment by keyboard for all user actions."?

Understanding Language of Page
in fact it is about the language being programmaticaly determined, so for example screen readers pronounce understandably.
The user cannot even see the indication.
Suggestion:

Objective
Assistive technology can identify the language of a page
Author task
Indicate the predominant language on a page through technology
Key beneficiaries
blind users and others reliant on speech output/dd>

Understanding Language of Parts
similar as for language of page
Suggestion:

Objective
Assistive technology can identify the languages used within a page<
Author task
Indicate when words are in a different language through technology
Key beneficiaries
blind users and others reliant on speech output/dd>

Understanding Non-text Content
I understand from the SC that the text alternative might be available only to assistive technology.
As a consequence, only users using assistive technology benefit.
Suggestion:
Benefeciaries: Users using assistive technology

Understanding Resize Text
Users with various vision impairments benefit from text enlargement. Also, low vision is a legal term in several countries, and most users benefiting from text enlargement without magnifying tool do not fall into the legal category.
Suggestion:
Beneficiaries: Users with reduced vision.

Understanding Use of Color
users benefiting most are color-blind users.
Only some of them might be low vision as well (in the legal sense).
Blind users do not benefit from visuals (unless they see but are legally blind).
Suggestion:
Color-blind users and those with reduced sight

@mbgower
Copy link
Contributor Author

mbgower commented Jun 12, 2023

Thanks @gundulaniemann
FYI, the better way to do this, if you are able, is to switch from the Conversation to the 'Files changed' view and click on the blue + symbol icon that appears at the line number (on hover or focus), as shown in this small screen snippet
image

This lets you put in comments directly on a file. Once in this mode, you can also activate the Add a suggestion button (Cmd+G) and directly enter new wording for that line.

This lets me put in responses directly into the file. Another benefit of following that procedure I describe is that it also appears in the Conversation thread. Unfortunately, only putting your comments in the Conversation view doesn't also get captured in the File view.

Following this procedure should also result in less typing for you, since you never need to say what file (or line) you're talking about. It's all baked in.

FYI, I've added in each of your comments, and then addressed and resolved them.

understanding/20/info-and-relationships.html Outdated Show resolved Hide resolved
understanding/20/keyboard.html Outdated Show resolved Hide resolved
understanding/20/language-of-page.html Outdated Show resolved Hide resolved
understanding/20/language-of-parts.html Outdated Show resolved Hide resolved
understanding/20/non-text-content.html Outdated Show resolved Hide resolved
understanding/20/resize-text.html Outdated Show resolved Hide resolved
understanding/20/use-of-color.html Outdated Show resolved Hide resolved
As per WG discussion on Jun 13, removing third line of In Brief.
I had originally planned on putting this in a separate pull request, but I realized it is much better for people to be able to see how the AAA differ from the A and AA criteria, where they are progressions
Added a word that improves readability and matches the AAA wording
Distinguishing between the AA and AAA requirements
aligning with changes from feedback
@mbgower mbgower changed the title in brief material for A and AA in brief material for WCAG 2.0 success criteria Jun 14, 2023
Altered in response to feedback
Modified in response to feedback
Added to distinguish from the AAA version. Welcome other suggestions. Is 'freehand drawing' or just 'drawing'
hopefully resolving conflicts for the files
@w3c w3c deleted a comment from detlevhfischer Jun 29, 2023
alastc and others added 2 commits July 7, 2023 16:57
Co-authored-by: Mike Gower <mikegower@gmail.com>
These changes comments by AGWG and EO reviewers
@@ -11,7 +11,7 @@ <h1>Understanding Name, Role, Value</h1>
<section id="brief">
<h2>In brief</h2>
<dl>
<dt>Objective</dt><dd>Authors implement components properly</dd>
<dt>Objective</dt><dd>People using Assistive Technology understand all components</dd>
Copy link

Choose a reason for hiding this comment

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

Is this needed? The AT needs to interpret the components properly so that the author's intent is conveyed to the user. However, when are people using AT ever required to even understand what a component is.

Copy link
Contributor Author

@mbgower mbgower Jul 10, 2023

Choose a reason for hiding this comment

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

@mgifford
It could come down to finding a better word than 'understand', but consider this scenario:

  1. when the author assigns the correct role, the AT identifies it as, say, a menu, and the user thinks "okay, I'm in a menu" She understands what the interaction is (and some ATs will tell her basic interaction keys).
  2. when the author assigns a name, the AT identifies it, and if the user is reliant on the AT presentation, she may think "Okay, this is the View menu, that's not the menu I want, so I'll proceed to the next one"
  3. when the author assigns the value or state, as the user advances, she hears 'View menu, closed', moves on to the Profile menu and might hear which profile is currently selected.

That's the intended outcome of 4.1.2, so arguably, the goal is about the user getting the information. This differs a bit from 4.1.1 Parsing, where it really is just about making sure the AT is able to understand/process the code, even if there's no discernible impact on the users these days (and the reason there's no real user benefit/effect is why 4.1.1 is being 'dropped').

matching up language for the 2 image of text criteria
@alastc
Copy link
Contributor

alastc commented Jul 31, 2023

Hi @mbgower, I was trying to merge, but there are some conflicts with the WCAG 2.2 understanding docs.

Given that we'll need to merge this into the WCAG 2.1 branch as well, could you remove the updates to the 2.2 docs please?

adding back Key beneficiaries line
@mbgower
Copy link
Contributor Author

mbgower commented Jul 31, 2023

@alastc as per email, I believe this has been resolved. Please LMK is you are still finding a conflict

Incorporation of feedback from EOWG
@alastc alastc dismissed stale reviews from detlevhfischer and dbjorge August 1, 2023 17:19

I think these have been incorporated

@alastc alastc merged commit 3d3c9f2 into main Aug 1, 2023
@alastc alastc deleted the wcag20QuickInfo branch August 1, 2023 17:21
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.

None yet

8 participants