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

Broken ARIA references Checklist Content Type in CMS #15814

Open
9 tasks
Tracked by #13563
laflannery opened this issue Oct 24, 2023 · 4 comments
Open
9 tasks
Tracked by #13563

Broken ARIA references Checklist Content Type in CMS #15814

laflannery opened this issue Oct 24, 2023 · 4 comments
Labels
a11y-defect-3 Moderate accessibility issue that should be fixed in the next 1-3 sprints accessibility Issues related to accessibility CMS Team CMS Product team that manages both editor exp and devops Defect Something isn't working (issue type) Drupal engineering CMS team practice area Needs refining Issue status Public Websites Scrum team in the Sitewide crew sitewide

Comments

@laflannery
Copy link
Contributor

laflannery commented Oct 24, 2023

Describe the defect

Within the edit screens on the Checklist content type, there are multiple aria-describedby attributes that do not correspond to an ID anywhere on the page.

  • Page Introduction
  • Calls to Action
  • VA Benefits Hubs
  • Related Information
  • Other Categories where people may look for this article

AC / Expected behavior

  • Determine what the original intention of the ARIA reference was/is. What additional context was it providing? Do we still need to provide that information in some other way?
  • UX to determine display of additional information if it is determined this is needed
  • Update attribute based on above determination (remove attribute or update it)
  • Accessibility review

Screenshots

Screenshot of the Page Introduction field error from AMP plugin. The aria-describedby attribute is pointing to something with the ID edit-field-intro-text-limited-html-0--description but if I search for that ID I can't find it anywhere on the page, it doesn't exist.

image

Screenshot of where the attribute is on the <div>
image

Accessibility Standard

WCAG version 2.0 A, Criterion 1.3.1

Team

Please check the team(s) that will do this work.

  • CMS Team
  • Public Websites
  • Facilities
  • User support
  • Accelerated Publishing
@laflannery laflannery added Defect Something isn't working (issue type) Needs refining Issue status CMS Team CMS Product team that manages both editor exp and devops accessibility Issues related to accessibility a11y-defect-2 Serious accessibility issue that should be fixed in the next 1-2 sprints labels Oct 24, 2023
@laflannery
Copy link
Contributor Author

I put the CMS label as a placeholder cause I'm not sure who this belongs to

@laflannery
Copy link
Contributor Author

@EWashb (or maybe @jilladams) are you able to tell me who owns Checklist, I can't tell from this mural who should be assigned this ticket

@laflannery laflannery added Drupal engineering CMS team practice area a11y-defect-3 Moderate accessibility issue that should be fixed in the next 1-3 sprints and removed a11y-defect-2 Serious accessibility issue that should be fixed in the next 1-2 sprints labels Nov 3, 2023
@laflannery
Copy link
Contributor Author

I saw a convo in slack that led me to putting this under PW ownership

@laflannery laflannery added Public Websites Scrum team in the Sitewide crew and removed CMS Team CMS Product team that manages both editor exp and devops labels Nov 6, 2023
@jilladams jilladams added the CMS Team CMS Product team that manages both editor exp and devops label Nov 7, 2023
@FranECross
Copy link

@laflannery We don't feel we'll be able to answer these questions/AC (below). Are you able to instead provide recommendations on what should be done?

  • Determine what the original intention of the ARIA reference was/is. What additional context was it providing? Do we still need to provide that information in some other way?
  • UX to determine display of additional information if it is determined this is needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-defect-3 Moderate accessibility issue that should be fixed in the next 1-3 sprints accessibility Issues related to accessibility CMS Team CMS Product team that manages both editor exp and devops Defect Something isn't working (issue type) Drupal engineering CMS team practice area Needs refining Issue status Public Websites Scrum team in the Sitewide crew sitewide
Projects
None yet
Development

No branches or pull requests

3 participants