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] Submission form's "serious" accessibility issues #1187

Open
tdonohue opened this issue May 12, 2021 · 3 comments
Open

[Deque Analysis] Submission form's "serious" accessibility issues #1187

tdonohue opened this issue May 12, 2021 · 3 comments
Assignees
Labels
accessibility bug claimed: 4Science 4Science team is working on this issue & will contribute back component: submission e/20 Estimate in hours funded Task is funded via the DSpace Development Fund 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 12, 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 Submission form as a total of 17 serious accessibility issues (not including color contrast which are handled in other tickets).

  1. (ID 469537) "<ul> and <ol> must only directly contain <li>, <script> or <template> elements" on the Subject field. Currently the Subject field (ds-chips) is implemented as a <ul> with a <div> and <input> in it. This is invalid.
  2. (ID 469707) "Status message is not automatically announced" on the "Processing... 91%" message that appears as a file is uploaded (in the drag & drop area).
  3. (ID 469871) "List or list item is not marked up properly." on the list of Communities/Collections within the Collection dropdown (at the top of the page).
  4. (ID 469915) "Element functions as if it has two roles." on each section header in the form (e.g. "Describe", "Describe", "Upload files", "Deposit license", etc).
    • NOTE from Deque: "The element has two roles a tab and a button, this can be confusing to the screen reader users."
  5. (ID 469928) "Non-decorative content is inserted using CSS pseudo-elements." affects several icons in the form : (1) draggable icon for reordering values, (2) delete icon for deleting a value, (3) status icons on sections -- green checkbox & orange exclamation.
    • NOTE: As of 7.6.1, this bug only appears to affect the "draggable icon for reordering values". The other icons mentioned above now have ARIA labels.
    • 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.
  6. (IDs 469971, 469981, 470003, 470008) "Multiple buttons have same name but different actions." occurs in several areas. In each area, we should add an aria-label or aria-describedby attribute to better describe the button/link.
    • Every trash can button next to multi-valued field values has the same name.
    • Every "Add more" link (under multi-valued fields) has the same name.
    • Every increment/decrement arrow in the "Date of Issue" field has the same name. Should be named something like "increment year", "increment month", etc.
    • Every delete button in "Upload files" section (after uploading multiple files) has the same name.
    • Every trash can button after editing an uploaded file has the same name
    • Every "Add more" link after editing an uploaded file has the same name.
  7. (IDs 469973, 470009, 470028) "Status message is not automatically announced" occurs on all error messages displayed below required fields (i.e. when form saves & a required field is not filled out, an error is displayed below that field). This also exists on embargo dates after selecting an embargo, and in the "You must accept the license" error message.
  8. (IDs 469975, 470055, 470019) "Label is not persistent" or "Visible label is missing" occurs in a few places
    • Date of Issue field's "Year/Month/Date" inputs do not have labels...they just have placeholder text (which disappears after you start typing)
    • Those same Date of Issue fields have invalid aria-label="name" attributes, which makes them even more confusing. (NO LONGER REPRODUCIBLE in 7.6.1)
    • Calendar popup in embargo date fields (after editing an uploaded file)... The Month and Date dropdowns have no label. (NO LONGER REPRODUCIBLE in 7.6.1)
  9. (ID 470054) "Content clipped at 200% zoom" on the placeholder text in the Date of Issued field's input fields for Year/Month/Date. Likely this could be fixed by removing the placeholder text in favor of a label.
  10. (ID 470057) "Content is overlapped at 320px width" on form buttons (at the bottom). When the form is viewed on a 320px device/width, the "Save", "Save for later", "Discard" buttons overlap all the form content. (As of 7.6.1, this overlapping no longer occurs. Buttons are in a separate, static footer. The style isn't great, but it ensures they do not overlap)

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=100&sortField=ordinal&sortDir=asc&filter%5Bseverity%5D=3&filter%5Btype%5D=issue&filter%5Bpage_number%5D=11&row=9

More Information / Tools

@tdonohue
Copy link
Member Author

Reopening, as only part of this has been fixed.

@tdonohue tdonohue reopened this Feb 28, 2023
@github-project-automation github-project-automation bot moved this from ✅ Done to 🏗 In Progress in DSpace 7.6 Release Feb 28, 2023
@tdonohue tdonohue moved this from 🏗 In Progress to 📋 To Do in DSpace 7.6 Release Mar 2, 2023
@atarix83
Copy link
Contributor

@tdonohue

we can take a look to those issues.
I'm not sure everything can be solved but making an analysis and fixing the issue could take around 20hrs i think

@tdonohue
Copy link
Member Author

Thanks, @atarix83 ! Assigning to you and adding the estimate.

@tdonohue tdonohue added claimed: 4Science 4Science team is working on this issue & will contribute back funded Task is funded via the DSpace Development Fund e/20 Estimate in hours and removed help wanted Needs a volunteer to claim to move forward Estimate TBD labels Sep 19, 2024
@tdonohue tdonohue moved this from 📋 To Do to 🏗 In Progress in DSpace 8.x and 7.6.x Maintenance Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility bug claimed: 4Science 4Science team is working on this issue & will contribute back component: submission e/20 Estimate in hours funded Task is funded via the DSpace Development Fund good first issue This is a smaller ticket/bug that may be easier for new contributors medium priority
Projects
Status: 🏗 In Progress
Development

No branches or pull requests

2 participants