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

Add aria-required to symptoms question in test card form #8552

Open
emyl3 opened this issue Feb 28, 2025 · 0 comments
Open

Add aria-required to symptoms question in test card form #8552

emyl3 opened this issue Feb 28, 2025 · 0 comments
Labels
Accessibility Issues that affect SR's accessibility Moderate Moderate bugs or a11y issues. Part of [Critical, Serious, Moderate] bug and a11y ranking labels.

Comments

@emyl3
Copy link
Collaborator

emyl3 commented Feb 28, 2025

Area or Component

Conduct tests -> Test Card -> "Select any symptoms the patient is experiencing" (for all diseases that ask this question) and "When did the patient's symptoms start?" (for non STI diseases AoE questions)

Quick Summary

For some conditionally required fields, when the user selects "Yes" for "Is the patient currently experiencing or showing signs of symptoms?", it is not clear that the following AoE questions are required:

  • "When did the patient's symptoms start?" (for non-STI diseases)
  • "Select any symptoms that the patient is experiencing" (for all diseases that ask this)

How to Replicate

See above

Accessibility Violation

aria-required-missing

Action Requested

Ensure those conditionally required fields have both a visual and programmatic way for users to know those fields are required. (with an * and a aria-required or required attribute.

Screenshots (optional)

Screenshot 2025-02-28 at 15.10.59.png

Screenshot 2025-02-28 at 15.18.25.png

Source Code (optional)

Original Deque Issue

@emyl3 emyl3 added the Accessibility Issues that affect SR's accessibility label Feb 28, 2025
@DanielSass DanielSass added this to the 508 resolution milestone Mar 7, 2025
@DanielSass DanielSass added the Moderate Moderate bugs or a11y issues. Part of [Critical, Serious, Moderate] bug and a11y ranking labels. label Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility Issues that affect SR's accessibility Moderate Moderate bugs or a11y issues. Part of [Critical, Serious, Moderate] bug and a11y ranking labels.
Projects
None yet
Development

No branches or pull requests

2 participants