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

Summary card actions must specify which card they are for #3673

Closed
Tracked by #3946
CharlotteDowns opened this issue May 23, 2023 · 1 comment · Fixed by #3961
Closed
Tracked by #3946

Summary card actions must specify which card they are for #3673

CharlotteDowns opened this issue May 23, 2023 · 1 comment · Fixed by #3961
Assignees
Labels
accessibility regulations failure Does not meet the Public Sector Accessibility Regulations (WCAG 2.2 level AA) accessibility statement These GitHub issues are or have previously been featured in our accessibility statement accessibility audit may 2023 Issues from May 2023 external accessibility audit against WCAG 2.2 criteria summary list
Milestone

Comments

@CharlotteDowns
Copy link
Contributor

CharlotteDowns commented May 23, 2023

This issue is from May 2023 external accessibility audit report.

WCAG Reference:
2.4.4 Link Purpose (In Context) (Level A)
Understanding Link Purpose (In Context) | How to Meet Link Purpose (In Context)

2.4.9 Link Purpose (Link Only) (Level AAA)
Understanding Link Purpose (Link Only) | How to Meet Link Purpose (Link Only)

Issue ID:
DAC_Link_Purpose_01

URLs:
https://govuk-frontend-pr-3503.herokuapp.com/full-page-examples/work-history#/delete

Screen shots

Summary card component with 'delete work history' link

The ‘Delete work history’ links for each section, rely on surrounding content to understand their context and which work they will be deleting. As the link is not marked up in the same list item, table cell, or using different methods of aria, the link requires a user to move their focus to understand which work history they will be deleting.

Current code ref(s)

#main-content > div > div > div:nth-child(2) > div.govuk-summary-card__title-wrapper > div > a

Screen-reader comments

“I located multiple links on the page which read to me as ‘Delete work history’. These links related to different sections of the page and would delete different areas although at the time of testing did not perform a function. I was not aware when browsing out of context that the links would perform a different action or what would be deleted as the links did not contain enough description within the link text to make this obvious. Giving each link text clear and unique information that can be understood either in or out of context will prevent any confusion and will ensure that I can select easily.”

Solution

Ensure that all links are descriptive of their purpose and functionality, allowing users of screen reading assistive technologies to confidently interact and understand the link.
The best way to make sure of this, is to locate links where some additional link context is needed to understand the purpose of the link. For each link:

  1. Check whether the context is contained in the same sentence, paragraph, list item, table cell, or associated table headers.
  2. Check whether the link context can be programmatically determined in some other way, for example, by using a WAI-ARIA property such as aria-label, aria-labelledby or aria-describedby on the link to provide sufficient context.

Consider adding additional text to the link which can be visually hidden to maintain the visual aesthetic.

@CharlotteDowns CharlotteDowns added accessibility audit may 2023 Issues from May 2023 external accessibility audit against WCAG 2.2 criteria labels May 23, 2023
@dav-idc
Copy link

dav-idc commented May 23, 2023

This issue has been assessed to not be a high-severity accessibility concern. It only meets 1 of our 3 criteria for a high-severity accessibility concern.

  • There is reasonable evidence that it makes it difficult or impossible for some people to complete certain tasks. ✅
    The evidence:
    • It is related to a WCAG level A failure.
    • It was assessed in an external accessibility audit as a 'high' priority accessibility concern.
  • It does not appear to affect the accessibility of essential services or critical infrastructure. ❌
  • There is no blocker to individual service teams efficiently resolving it on their own. ❌

As such, this will not be flagged as a high-priority issue.

In the coming weeks we will investigate whether it fails WCAG level A criterion: 2.4.4 Link Purpose (In Context).

@dav-idc dav-idc added the accessibility concern Bug, feature request or question about the accessibility of a portion of a product (not a WCAG fail) label May 23, 2023
@querkmachine querkmachine self-assigned this Jul 24, 2023
@querkmachine querkmachine moved this from Backlog 🗄 to Needs review 🔍 in GOV.UK Design System cycle board Jul 24, 2023
@querkmachine querkmachine moved this from Needs review 🔍 to Ready to release 🚀 in GOV.UK Design System cycle board Jul 27, 2023
@CharlotteDowns CharlotteDowns added the accessibility regulations failure Does not meet the Public Sector Accessibility Regulations (WCAG 2.2 level AA) label Aug 1, 2023
@dav-idc dav-idc added the accessibility statement These GitHub issues are or have previously been featured in our accessibility statement label Sep 5, 2023
@36degrees 36degrees added this to the v5.0 milestone Jan 26, 2024
@36degrees 36degrees moved this from Ready to release 🚀 to Done 🏁 in GOV.UK Design System cycle board Jan 26, 2024
@selfthinker selfthinker removed the accessibility concern Bug, feature request or question about the accessibility of a portion of a product (not a WCAG fail) label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility regulations failure Does not meet the Public Sector Accessibility Regulations (WCAG 2.2 level AA) accessibility statement These GitHub issues are or have previously been featured in our accessibility statement accessibility audit may 2023 Issues from May 2023 external accessibility audit against WCAG 2.2 criteria summary list
Projects
Development

Successfully merging a pull request may close this issue.

5 participants