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

Accessibility Insights report still shows "..tests that cover all the WCAG 2.1 AA success criteria" instead of WCAG 2.2 #7209

Open
jasvindargillQA opened this issue Jan 19, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@jasvindargillQA
Copy link

Latest version of A11y Insights have WCAG 2.2 criteria tests listed in full assessment. However when we generate the report it still says WCAG 2.1.

Sample heading from report:
"This report shows the overall accessibility of the website or web app through a combination of automated and manual tests that cover all the WCAG 2.1 AA success criteria."

@jasvindargillQA jasvindargillQA added the bug Something isn't working label Jan 19, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the status: new This issue is new and requires triage by DRI. label Jan 19, 2024
@DaveTryon DaveTryon added active (Dev) The assigned dev is working to perform the work. and removed status: new This issue is new and requires triage by DRI. labels Jan 30, 2024
@DaveTryon DaveTryon self-assigned this Jan 30, 2024
DaveTryon added a commit that referenced this issue Jan 31, 2024
#### Details

As called out in #7209, when we added WCAG 2.2 tests, we still had a few
places in the UX where we referenced "WCAG 2.1 AA success criteria".
When we discussed this in triage, the approved updated wording is now
"WCAG 2.1 AA and 2.2 AA success criteria". While #7209 was specific to
the report, this changes 3 other places in the UX.

##### Screenshots

The text size of the elements has not changed, but GitHub is scaling the
screen grabs that are slightly different sizes, hence the potential for
these seeming to be different sizes:

Element | Old | New
--- | --- | ---
Assessment Overview | <img width="405" alt="image"
src="https://github.com/microsoft/accessibility-insights-web/assets/45672944/f284af96-13f4-4705-a109-b66c4d6acf3b">
| <img width="401" alt="image"
src="https://github.com/microsoft/accessibility-insights-web/assets/45672944/6214ee17-4d5d-48c4-9a65-740262c033e1">
QuickAssess Overview Help | <img width="210" alt="image"
src="https://github.com/microsoft/accessibility-insights-web/assets/45672944/8f819126-a85f-4a7d-838b-471c37cb5105">
| <img width="208" alt="image"
src="https://github.com/microsoft/accessibility-insights-web/assets/45672944/9793ce6e-b44f-4a16-8bd7-bf723f7fedf7">
Assessment Report | <img width="586" alt="image"
src="https://github.com/microsoft/accessibility-insights-web/assets/45672944/33e009ed-acff-46b0-88c8-710cf11a694c">
| <img width="578" alt="image"
src="https://github.com/microsoft/accessibility-insights-web/assets/45672944/2c33174c-feb5-4b1f-8a45-fb93f29e54f1">
QuickAsses Report | <img width="581" alt="image"
src="https://github.com/microsoft/accessibility-insights-web/assets/45672944/77184ac8-54d6-40a4-b94c-675cf2e39ace">
| <img width="575" alt="image"
src="https://github.com/microsoft/accessibility-insights-web/assets/45672944/d779c738-d162-4274-83d7-079850159c7e">

##### Motivation

Address #7209

##### Context

<!-- Are there any parts that you've intentionally left out-of-scope for
a later PR to handle? -->

<!-- Were there any alternative approaches you considered? What
tradeoffs did you consider? -->

#### Pull request checklist
<!-- If a checklist item is not applicable to this change, write "n/a"
in the checkbox -->
- [x] Addresses an existing issue: #7209
- [x] Ran `yarn fastpass`
- [x] Added/updated relevant unit test(s) (and ran `yarn test`)
- [x] Verified code coverage for the changes made. Check coverage report
at: `<rootDir>/test-results/unit/coverage`
- [x] PR title *AND* final merge commit title both start with a semantic
tag (`fix:`, `chore:`, `feat(feature-name):`, `refactor:`). See
`CONTRIBUTING.md`.
- [x] (UI changes only) Added screenshots/GIFs to description above
- [x] (UI changes only) Verified usability with NVDA/JAWS
@DaveTryon DaveTryon removed their assignment Jan 31, 2024
@DaveTryon DaveTryon removed the active (Dev) The assigned dev is working to perform the work. label Jan 31, 2024
@jasvindargillQA jasvindargillQA changed the title Accessibility Insights report sill shows "..tests that cover all the WCAG 2.1 AA success criteria" instead of WCAG 2.2 Accessibility Insights report still shows "..tests that cover all the WCAG 2.1 AA success criteria" instead of WCAG 2.2 Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Needs release
Development

No branches or pull requests

2 participants