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

Esc key does not dismiss dialog opened with sp-dialog-wrapper #1036

Closed
davidclark87 opened this issue Dec 4, 2020 · 2 comments · Fixed by #1114
Closed

Esc key does not dismiss dialog opened with sp-dialog-wrapper #1036

davidclark87 opened this issue Dec 4, 2020 · 2 comments · Fixed by #1114

Comments

@davidclark87
Copy link
Collaborator

Expected Behaviour

Pressing Esc key should close a modal dialog. Since the dialog could be opened via keyboard, it should be dismissable via keyboard.

Actual Behaviour

Pressing Esc has no effect. Workaround is to click the X in the top right corner or outside of the dialog area to dismiss it.

Reproduce Scenario (including but not limited to)

Steps to Reproduce

  1. Go to https://opensource.adobe.com/spectrum-web-components/components/dialog-wrapper
  2. Click the first Toggle Dialog button on the page (under the "Small" Example)
  3. With the dialog now open, try pressing Esc. Nothing happens
    image

Platform and Version

Tested on Chrome 87 (Mac)

Sample Code that illustrates the problem

See docs

Logs taken while reproducing problem

N/A

@Westbrook
Copy link
Contributor

This is outlined in #710 and #866, but the functionality here exists, but is not yet being used in the documentation site for this element. Great to have it called out so clearly, however, so thanks!

@davidclark87
Copy link
Collaborator Author

davidclark87 commented Dec 4, 2020

@Westbrook sorry I'm not following - doesn't this behavior on the documentation site imply the component has an a11y issue, at least when configured with the options shown?

Is the point that those options are actually invalid or deprecated with respect to the current component API?

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

Successfully merging a pull request may close this issue.

2 participants