Skip to content

DRYD-2130: Accessibility > Missing form label (Criteria 3.3.2) - #27

Merged
spirosdi merged 3 commits into
mainfrom
bugfix/dryd-2130
Jul 16, 2026
Merged

DRYD-2130: Accessibility > Missing form label (Criteria 3.3.2)#27
spirosdi merged 3 commits into
mainfrom
bugfix/dryd-2130

Conversation

@spirosdi

@spirosdi spirosdi commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

What does this do?

It fixes all the missing form label a11y issues. The main changes are:

  • Forward id to the read-only inputs rendered by DateInput, DateTimeInput, DropdownMenuInput, and IDGeneratorInput, so labels remain associated in read-only views
  • Forward aria-label/aria-labelledby to the read-only input rendered by DropdownMenuInput
  • Add keywordInputLabel, recordTypeInputLabel, and vocabularyInputLabel props to QuickSearchInput, applied as aria-label to the inputs
  • Add aria-labelledby to RepeatingInput instance inputs, pointing to the group header label
  • Add renderAriaLabel prop to TabularCompoundInput, giving row inputs in every repeating instance an aria-label
  • Add ids to the inputs of UploadInput, associating the Upload, File, and URL labels. FileInput now places its id on the file input

Why are we doing this? (with JIRA link)
We need to fix the Missing form label (Criteria 3.3.2) across the application: https://collectionspace.atlassian.net/browse/DRYD-2130

How should this be tested? Do these changes have associated tests?
Please follow the steps in: collectionspace/cspace-ui.js#361

Dependencies for merging? Releasing to production?
no dependencies

Has the application documentation been updated for these changes?
changelog has been updates

Did someone actually run this code to verify it works?
@spirosdi ran it locally

@spirosdi
spirosdi merged commit ea3a5b2 into main Jul 16, 2026
5 checks passed
@spirosdi
spirosdi deleted the bugfix/dryd-2130 branch July 16, 2026 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants