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

Deposit form: "major" accessibility issues that may not need design #3170

Closed
7 of 12 tasks
ndushay opened this issue Jun 29, 2023 · 0 comments
Closed
7 of 12 tasks

Deposit form: "major" accessibility issues that may not need design #3170

ndushay opened this issue Jun 29, 2023 · 0 comments
Assignees
Labels
2023 Summer 2023 workcycle accessibility accessibility improvements

Comments

@ndushay
Copy link
Contributor

ndushay commented Jun 29, 2023

From https://docs.google.com/spreadsheets/d/1Fs5sY1zgpovWVdHMqpJWRu2Zvyr53jw7, here are "major" accessibility issues for the Deposit Form that we can fix without design changes. Note that I addressed accessibility level A and AA problems noted by the SiteImprove, WAVE and Axe browser plugins in PR #3169.

Note that solutions may be suggested in https://docs.google.com/spreadsheets/d/1Fs5sY1zgpovWVdHMqpJWRu2Zvyr53jw7.

These can be separate PRs, and/or split into separate tickets.

  • There are multiple "Remove" buttons that perform different actions. Form field labels should be unique. See Deposit form remove button accessibility #3181
  • When the "Upload one or more files (more than 10GB) which will be displayed as uploaded, including any file hierarchy" option is selected, the six steps listed are not structured as a list. See Switch upload instructions to a list #3333
  • When the "See more options" control is activated, focus moves past the new content to the "Continue" button (in both the modal dialog and in the main page).
  • When the "See more options"/"See fewer options" disclosure widget is used it is missing an aria-expanded (in the modal dialog and on the main page).
  • There are headings that are only styled to look like headings but are not marked as such in HTML. Each section has a visible heading, but they are not marked up as headings. Examples include:
    • Add your files*
    • Title of deposit and contact information*
    • List authors and contributors *
      and many more.
  • There are groups of form controls that do not provide and elements. The radio buttons in "Add your files*" and the date fields in "Enter dates related to your deposit" are not grouped appropriately.
  • Form field constraints and/or errors are either not clearly displayed or are not programmatically associated with the field they apply to. Errors are not explicitly associated with form fields, which means that the error text is not conveyed when the fields have focus and are in error.
    There are <select> elements with asterisk that are not programmatically conveyed as required:
    - In the List authors and contributors section, the Role term <select>
    - In the Select a license section, the License <select>
  • The toggle for "Use auto-generated citation" does not receive keyboard focus: PR Visually indicate when the form buttons get focus #3335
  • The "Remove" buttons (trash can images) receive keyboard focus with no visual indication when using TAB or SHIFT+TAB to navigate through them. PR Visually indicate when the form buttons get focus #3335
  • There are numerous instances of placeholder links, elements without an href. However, they are receiving focus because tabindex="0" has been added which means they are not acting as placeholder links. Each of these lacks a role. Custom controls should provide textual name (link or button label, form label text), role (i.e. identify as button, link, text field, etc.), and/or state information (i.e. "expanded", "collapsed", selected, current). Possible fix shown in PR Improve accessibility for collections pages #3161; Same problem listed in Collection View - Deposits page: "major" accessibility issues that may not need design #3176
  • the 4 errors noted by WAVE (I could not find the elements to fix)
    image
  • (Minor level) The aria-expanded attribute is not allowed on a <div> element.
@ndushay ndushay changed the title Deposit form: accessibility review stuff not noted by plugins Deposit form: "major" accessibility review issues not noted by plugins (or I wasn't able to find them) Jun 29, 2023
@ndushay ndushay changed the title Deposit form: "major" accessibility review issues not noted by plugins (or I wasn't able to find them) Deposit form: "major" accessibility review issues remaining and not needing design Jun 29, 2023
@ndushay ndushay added the 2023 Summer 2023 workcycle label Jun 29, 2023
@ndushay ndushay changed the title Deposit form: "major" accessibility review issues remaining and not needing design Deposit form: "major" accessibility issues that may not need design Jun 30, 2023
@ndushay ndushay added the accessibility accessibility improvements label Jul 5, 2023
justinlittman added a commit that referenced this issue Jul 5, 2023
justinlittman added a commit that referenced this issue Jul 6, 2023
@aaron-collier aaron-collier self-assigned this Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2023 Summer 2023 workcycle accessibility accessibility improvements
Projects
None yet
Development

No branches or pull requests

2 participants