Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove deprecated code, add strict and update jquery to latest version #253

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Remove deprecated code, add strict and update jquery to latest version #253

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jul 26, 2019

I have done three main things:

  1. Removed the a.keyCode==37 and a.keyCode==39

  2. Added strict to those two strings

e.g. in to this:

a.key === 'ArrowLeft'
a.key === 'ArrowRight'
  1. Updated your jQuery to latest version because the one you are using has way too many XSS attacks vuls.

Linked to this github issue: #252

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants