Skip to content
This repository has been archived by the owner on May 24, 2024. It is now read-only.

[terra-pills] Fixed JAWS(SR) providing navigation instructions on empty pill box #1935

Merged
merged 7 commits into from
Dec 8, 2023

Conversation

gt106551
Copy link
Contributor

@gt106551 gt106551 commented Dec 7, 2023

Summary

JAWS providing navigation instructions on empty pill box has context menu

What was changed:
Changed role="list" to role={navigator.userAgent.indexOf('Edg') !== -1 ? '' : 'list'}.

Why it was changed:
To fix JAWS providing navigation instructions on empty pill box

Testing

This change was tested using:

  • WDIO
  • Jest
  • Visual testing (please attach a screenshot or recording)
  • Other (please describe below)
  • No tests are needed

Reviews

In addition to engineering reviews, this PR needs:

  • UX review
  • Accessibility review
  • Functional review

Additional Details

This PR resolves:

UXPLATFORM-9907


Thank you for contributing to Terra.
@cerner/terra

@gt106551 gt106551 changed the title JAWS providing navigation instructions on empty pill box has context menu [terra-pills] Fixed JAWS(SR) providing navigation instructions on empty pill box has context menu Dec 7, 2023
@gt106551 gt106551 changed the title [terra-pills] Fixed JAWS(SR) providing navigation instructions on empty pill box has context menu [terra-pills] Fixed JAWS(SR) providing navigation instructions on empty pill box Dec 7, 2023
@github-actions github-actions bot temporarily deployed to preview-pr-1935 December 8, 2023 08:32 Destroyed
@supreethmr supreethmr merged commit a7dc21d into main Dec 8, 2023
22 checks passed
@supreethmr supreethmr deleted the terra-pills-jaws-issue branch December 8, 2023 09:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants