Clarity: Modal produces an error when pressing ESC key

Created on 22 Sep 2017  路  6Comments  路  Source: vmware/clarity

Select one ... (check one with "x")

[x] bug
[ ] feature request
[ ] enhancement

Expected behavior


An html with a modal component should not produce errors when pressing the ESC key.

Actual behavior


When the modal is hidden, when pressing the ESC key, it's generating an error:

image

Reproduction of behavior


  • Include a link to the reproduction scenario you created by forking one of the Clarity Plunker Templates:

Environment details

  • Angular version: 4.3.6

  • Clarity version: 0.10.6

  • OS and version:

  • Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

bug

Most helpful comment

@mathisscott this issue is related to https://github.com/vmware/clarity/pull/1472
In this case, it should be putting focus onto the body.

I'll look into it.

All 6 comments

Hi @whizkidwwe1217
I know how much you 鉂わ笍 plunkr. But could you replicate your issue in one this time?

I've tried reproducing this on both the website and plunkr and cannot.

Hi @mathisscott
You can definitely replicate it with the default plunkr. To replicate it, don't open up or show the modal. Just run the plunkr and hit the ESC key. :-)
image

Ah. Got it. I was opening the modal first.

I'll try replicating it again on Monday. Thanks!

@mathisscott Thanks!

@mathisscott this issue is related to https://github.com/vmware/clarity/pull/1472
In this case, it should be putting focus onto the body.

I'll look into it.

Hi there 馃憢, this is an automated message. To help Clarity keep track of discussions, we automatically lock closed issues after 14 days. Please look for another open issue or open a new issue with updated details and reference this one as necessary.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

elesueur picture elesueur  路  3Comments

nkolchakov picture nkolchakov  路  3Comments

JohannesRudolph picture JohannesRudolph  路  4Comments

BugsyFTW picture BugsyFTW  路  3Comments

clane picture clane  路  3Comments