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

fix: don't make aria-live containers inert #293

Merged
merged 2 commits into from
Mar 28, 2020
Merged

fix: don't make aria-live containers inert #293

merged 2 commits into from
Mar 28, 2020

Conversation

greatislander
Copy link
Collaborator

@greatislander greatislander commented Mar 28, 2020

Description

The dialog component sets direct descendants of the body to inert, which interferes with @wordpress/a11y and the aria-live containers it uses. This PR ensures that aria-live containers are not set to inert.

Steps to test

When invoking a dialog, direct descendants of the body tag with an aria-live attribute should not be set to inert.

Additional information

Not applicable.

Related issues

@greatislander greatislander self-assigned this Mar 28, 2020
@greatislander greatislander added this to the 1.0.0-alpha.12 milestone Mar 28, 2020
@greatislander greatislander added the enhancement New feature or request label Mar 28, 2020
@greatislander greatislander requested a review from jhung March 28, 2020 16:22
@netlify
Copy link

netlify bot commented Mar 28, 2020

Deploy preview for pinecone ready!

Built with commit ee5206e

https://deploy-preview-293--pinecone.netlify.com

@greatislander greatislander merged commit b12d4fe into dev Mar 28, 2020
@greatislander greatislander deleted the fix/inert branch March 28, 2020 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant