Closed EdwinGuzman closed 3 months ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
nypl-design-system | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Aug 27, 2024 8:35pm |
Fixes JIRA ticket DSD-1823
This PR does the following:
MultiSelect
'sbuttonText
prop value as the internalCheckboxGroup
's label text value. The previous hardcoded value had no meaning, was repetitive, and was confusing. This updates gives it more context for screen readers.div
inside the accordion button to aspan
for better HTML DOM.How has this been tested?
Accessibility concerns or updates
Accessibility Checklist
aria-live
is used, a screenreader was used to verify the text is read.ref
s, focus management was reviewed.Open Questions
-
Checklist:
Front End Review: