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.

Sign up for GitHub

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

Jump to bottom

e.key is not defined on Chrome 47 #65

Closed
hannimator opened this issue Jun 28, 2017 · 7 comments
Closed

e.key is not defined on Chrome 47 #65

hannimator opened this issue Jun 28, 2017 · 7 comments
Labels
bug

Comments

@hannimator
Copy link

Using chrome 47.0.2526.80 and e.key is not defined.
@oriondean oriondean added the bug label Jun 28, 2017
@rwilliams-scottlogic
Copy link

https://caniuse.com/#feat=keyboardevent-key
available since Chrome 51, May 2016

@rwilliams-scottlogic
Copy link

Bug makes a case for doing Browserstack #33

@ryanggrey
Copy link
Contributor

Might also be wise for us to be clear about our browser support policy and publicise via README.

@ryanggrey
Copy link
Contributor

ryanggrey commented Jun 29, 2017 •
edited
Loading

This can hopefully be used for browsers that don't support event.key - https://www.npmjs.com/package/keyboardevent-key-polyfill. I'll add details to README.

@ryanggrey
Copy link
Contributor

PR #69

@ryanggrey
Copy link
Contributor

@hannimator Are we good to close this issue given the polyfill workaround for older browsers?

@hannimator
Copy link
Author

Sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ryanggrey @oriondean @rwilliams-scottlogic @hannimator