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 Issue in Media content pagination #12912

Open
1 of 7 tasks
Tracked by #13563
ndouglas opened this issue Mar 9, 2023 · 7 comments
Open
1 of 7 tasks
Tracked by #13563

Accessibility Issue in Media content pagination #12912

ndouglas opened this issue Mar 9, 2023 · 7 comments
Labels
a11y-defect-2 Serious accessibility issue that should be fixed in the next 1-2 sprints accessibility Issues related to accessibility CMS Team CMS Product team that manages both editor exp and devops

Comments

@ndouglas
Copy link
Contributor

ndouglas commented Mar 9, 2023

Description

Still an issue in D10

Seeing this Axe error:

Screenshot 2023-03-09 at 2 45 05 PM

for this item:

Screenshot 2023-03-09 at 2 44 36 PM

on /admin/content/media.

Todos

Team

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

  • CMS Team
  • Public Websites
  • Facilities
  • User support

Previous Team Points

3

@ndouglas ndouglas added Needs refining Issue status CMS Team CMS Product team that manages both editor exp and devops accessibility Issues related to accessibility labels Mar 9, 2023
@laflannery
Copy link
Contributor

I looked into this and this is done pretty much everywhere as a very established pattern (Drupal, VA, USWDS). I have a proposed solution but I'm running it by the other a11y folks just because I wanted some other eyes.

@ndouglas For now, if this is blocking you or others from completing tests or merging code I do not have an issue with you writing an ignore in for this since I have this documented and as it's upstream might take a bit to get resolved. If you do, can you note it in here so we can remember to remove it later?

@laflannery laflannery added Needs refining Issue status and removed Needs refining Issue status labels Mar 10, 2023
@ndouglas
Copy link
Contributor Author

Yeah, that works for me. Thank you!

@laflannery
Copy link
Contributor

@productmike
Copy link

@tonytaylor @edmund-dunn If able to a deeper look at the patch and update story point as necessary. F/up 7/5

@productmike
Copy link

@edmund-dunn @tonytaylor
Can y'all include this in your pre-finement discussions prior to next week's refinement session?

cc: @laflannery @BerniXiongA6

@laflannery laflannery added the a11y-defect-2 Serious accessibility issue that should be fixed in the next 1-2 sprints label Aug 24, 2023
@ndouglas
Copy link
Contributor Author

Edmund suggests this should be prioritized post-D10.

@laflannery laflannery removed the Needs refining Issue status label Aug 25, 2023
@laflannery laflannery removed their assignment Nov 6, 2023
@srancour
Copy link
Contributor

Still an issue and should be addressed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-defect-2 Serious accessibility issue that should be fixed in the next 1-2 sprints accessibility Issues related to accessibility CMS Team CMS Product team that manages both editor exp and devops
Projects
None yet
Development

No branches or pull requests

4 participants