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

JAWS - heading in skipped in popup dialog box #52

Open
UiTeamIntegra opened this issue Dec 29, 2020 · 2 comments
Open

JAWS - heading in skipped in popup dialog box #52

UiTeamIntegra opened this issue Dec 29, 2020 · 2 comments

Comments

@UiTeamIntegra
Copy link

Steps to recreate:

  1. access page Link
  2. select any --> modal open modal
  3. navigate using key down button through the dialog box

Expected result:
Focus should be available in the 'Title' heading

Actual result:
Focus skips 'Title' heading

Environment
OS Version : Windows10 Windows8
JAWS Version : 2020.2004.66 ILM
Chrome Version : 87

@StommePoes
Copy link

Note to devs: "Focus" should not be available (title should NOT have tabindex), however when browsing in JAWS using the DownArrow key, the title should be readable like any other content.

@StommePoes
Copy link

StommePoes commented Jan 2, 2021

Devs: See related ticket #53

The issue is that all versions of the modal (excepting the last one "Open modal that wrapps app instead of creating wrapper div") are inside the body element which has aria-hidden="true".

This means everything inside the web page is deliberately hidden from assistive technologies, and is not how modal dialogs should be created. This is a very very old react-modal bug which I thought had been fixed years ago.

aria-hidden must be set on something containing all the web page content except the modal. Setting aria-hidden on an ancestor of the modal means the modal is also hidden.

@UiTeamIntegra for now when creating modal dialogs with this component, restrict yourselves to the "Open modal that wrapps app instead of creating wrapper div" version. This is the only one which is coded correctly.

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

No branches or pull requests

2 participants