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

[Deque Analysis] Item Edit "Metadata" Tab "serious" accessibility issues #1193

Closed
tdonohue opened this issue May 14, 2021 · 3 comments · Fixed by #3155
Closed

[Deque Analysis] Item Edit "Metadata" Tab "serious" accessibility issues #1193

tdonohue opened this issue May 14, 2021 · 3 comments · Fixed by #3155
Assignees
Labels
accessibility bug component: Item (Archived) Item display or editing good first issue This is a smaller ticket/bug that may be easier for new contributors medium priority

Comments

@tdonohue
Copy link
Member

tdonohue commented May 14, 2021

Serious: This issue results in serious barriers for people with disabilities, and will partially prevent them from accessing fundamental features or content. People relying on assistive technologies will experience significant frustration as a result. Issues falling under this category are major problems, and remediation should be a priority.

Deque Analysis Summary

Our Item Edit "Metadata" tab has a total of 3 serious accessibility issues (not including color contrast which are handled in other tickets).

  1. (ID 471438) "Non-decorative content is inserted using CSS pseudo-elements." on all buttons. Every button on this page (edit, delete, stop editing, undo) is represented by an icon, but those icons are inserted using CSS :before and :after pseudo-element.
    • RULE : Non-decorative content MUST NOT be inserted using CSS :before and :after pseudo-elements unless there is a way to access the content with CSS turned off.
    • HOW TO FIX: Fix this issue by including the content directly in the DOM using HTML
    • NOTE: This same issue occurs with all buttons on "Bitstreams" tab, so this fix should use the same solution as [Deque Analysis] Item Edit "Bitstreams" Tab "serious" accessibility issues #1191
  2. (ID 471439) "Buttons have same name but different actions." Every button on this page has no name/label (edit, delete, stop editing, undo). Their text should either be in a <button> tag and/or have an aria-label or similar.
  3. (ID 471443) "Active user interface component lacks 3 to 1 contrast ratio." on the Language input field (when editing a row in the table). The visual boundary of an ACTIVE user interface component does not have at least a 3 to 1 luminosity contrast ratio with either the inner or outer adjacent background.
    • On white background, Visual boundary color: #CED4DA, Adjacent background color: #FFFFFF, Contrast ratio: 1.5:1
    • On grayish background, Visual boundary color: #E8EBF3, Adjacent background color: #FFFFFF, Contrast ratio: 1.2:1
    • FIXED BY Deque Analysis Color Contrast fixes #2587

Full list of issues is viewable at (requires login): https://axeauditor.dequecloud.com/test-run/0856438a-a19a-11eb-bc31-b7d5be387c86/issues?activeTab=dt-issue&page=0&pageSize=25&sortField=ordinal&sortDir=asc&filter%5Bseverity%5D=3&filter%5Btype%5D=issue&filter%5Bpage_number%5D=14&row=9

More Information / Tools

@tdonohue tdonohue added this to the 7.0 milestone May 14, 2021
@tdonohue tdonohue modified the milestones: 7.0, 7.1 Jul 28, 2021
@tdonohue tdonohue removed this from the 7.1 milestone Oct 27, 2021
@tdonohue tdonohue added help wanted Needs a volunteer to claim to move forward good first issue This is a smaller ticket/bug that may be easier for new contributors labels May 20, 2022
@tdonohue tdonohue moved this to To Do in DSpace 7.5 release Oct 11, 2022
@tdonohue tdonohue moved this to 📋 To Do in DSpace 7.6 Release Feb 27, 2023
@Andrea-Guevara
Copy link
Contributor

Good morning @tdonohue! Following the note/suggestion on how to fix the accessibility problem of the item with id 471438, I would like to contribute to this activity. As I understand it, the problem was solved by adding the "aria-label" attribute to the buttons. Is that what this is about?

@tdonohue
Copy link
Member Author

tdonohue commented Jul 1, 2024

@Andrea-Guevara : I took a closer look at this today, and I believe you are correct. It's just the aria-label that is missing from these buttons. That will ensure they are "visible" to screen readers even when CSS is completely disabled.

I'll assign this to you. Thanks!

@tdonohue tdonohue removed help wanted Needs a volunteer to claim to move forward Estimate TBD labels Jul 1, 2024
@tdonohue tdonohue moved this from 📋 To Do to 🏗 In Progress in DSpace 8.x and 7.6.x Maintenance Jul 1, 2024
Andrea-Guevara pushed a commit to Neki-IT/dspace-angular that referenced this issue Jul 1, 2024
…e to the add, save, discard and undo buttons on the metadata editing page
@Andrea-Guevara
Copy link
Contributor

@tdonohue: Good afternoon! Thanks for the feedback, I've made a PR with the changes. I'm available to answer any questions you may have.

#3155

@github-project-automation github-project-automation bot moved this from 🏗 In Progress to ✅ Done in DSpace 8.x and 7.6.x Maintenance Jul 10, 2024
github-actions bot pushed a commit that referenced this issue Jul 10, 2024
…he add, save, discard and undo buttons on the metadata editing page

(cherry picked from commit 4e783e7)
github-actions bot pushed a commit that referenced this issue Jul 10, 2024
…he add, save, discard and undo buttons on the metadata editing page

(cherry picked from commit 4e783e7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility bug component: Item (Archived) Item display or editing good first issue This is a smaller ticket/bug that may be easier for new contributors medium priority
Projects
Development

Successfully merging a pull request may close this issue.

3 participants