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

Feedback accordion should indicate expanded/collapsed state #2468

Closed
alundgard opened this issue Jul 16, 2024 · 2 comments
Closed

Feedback accordion should indicate expanded/collapsed state #2468

alundgard opened this issue Jul 16, 2024 · 2 comments
Assignees
Labels

Comments

@alundgard
Copy link
Member

alundgard commented Jul 16, 2024

Current behavior

The Feedback accordion does not indicate to screen readers when it is "collapsed" or "expanded".

Steps to reproduce

Example page: Martin Wong Catalogue Raisonné, or any other exhibit.

Use a screen reader (such as VoiceOver) to navigate the Feedback link in the page header. Clicking the link gives no indication that a feedback form expands and is available for interaction on the page.

Possible solution

WCAG Level A: Understanding Success Criterion 4.1.2: Name, Role, Value.

See the WAI Accordion Example.

Note: This issue is not specific to Spotlight. It also appears in Arclight Beta, Virtual Tribunals, and possibly other applications.

Note: Ongoing discussions with the design team (DLUX) suggest replacing the Feedback accordion with a modal, which would obviate the need to fix this issue. This is part of a broader effort to redesign DLSS application headers.

Reported by SODA

Violation: Ensure custom controls provide proper textual name, role, and state information
Severity: Major

[Issue]
The feedback link does not indicate that it opens an 'accordion' style portion of the screen, nor does it indicate the current state of open or closed.

[User Impact]
When controls do not provide name, role and/or state, screen reader users will not know their purpose and current state (if applicable).

[Recommendation]
Follow the WAI pattern for accordions. The control element should have aria-expanded and aria-controls set based on the state.

https://www.w3.org/WAI/ARIA/apg/patterns/accordion/examples/accordion/

@alundgard alundgard added the a11y label Jul 16, 2024
@jcoyne
Copy link
Contributor

jcoyne commented Jul 16, 2024

If we get to bootstrap 5, then we can use the feedback accordion in the component library: https://sul-dlss.github.io/component-library/header/.

@jcoyne jcoyne self-assigned this Dec 5, 2024
@jcoyne
Copy link
Contributor

jcoyne commented Dec 19, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants