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

UIPBEX-50 #127

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

UIPBEX-50 #127

wants to merge 3 commits into from

Conversation

chris-hellen
Copy link
Contributor

Purpose

Added dynamic accessibility label's to all select elements so that screen readers can get an accurate picture of intended use. This is necessary so that blind/visually impaired users are able to understand and interact with the page.

[JIRA Issue](https://folio-org.atlassian.net/browse/UIPBEX-50 atlOrigin=eyJpIjoiM2E4MjkzMzg5ZmY1NDNiNmFlZWQ2YjEzN2FjZTkyZjYiLCJwIjoiaiJ9)

Approach

Fetched the selected choice from the select element and passed that value to the 'aria-label' prop. Added label elements so that there were no visible changes. Adapted translations so that internationalization would still function.

@chris-hellen chris-hellen requested a review from danetsao May 28, 2024 17:47
Copy link

Jest Unit Test Results

  1 files  ±0   73 suites  ±0   4m 12s ⏱️ +3s
401 tests ±0  401 ✅ ±0  0 💤 ±0  0 ❌ ±0 
439 runs  ±0  439 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 70c9b77. ± Comparison against base commit c398db4.

Copy link

@chris-hellen chris-hellen requested a review from ncovercash June 10, 2024 19:00
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.

1 participant