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

Upgrade to Vue 3 #48

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

semiaddict
Copy link

@semiaddict semiaddict commented Feb 27, 2022

This PR upgrades the entire package to work with Vue 3, and replaces the usage of the deprecated KeyboardEvent.keycode property by KeyboardEvent.key.

Since Vue 3 is now officially out and the default at vuejs.org, it might be interesting to move the original master branch to a vue2 branch, and make the version in the PR the default one.

Closes #31, #39

This is mostly important when teleporting a component to another window/iframe
@zcuric
Copy link
Collaborator

zcuric commented Mar 1, 2022

LGTM 👍🏻 Awesome job!
@vladimyr @Dafrok can you take a look?

@semiaddict
Copy link
Author

@vladimyr @Dafrok is there anything else I can do to get this checked and merged ?

@AlvesJorge
Copy link

Ping! @vladimyr @Dafrok (On the off-chance that this PR got buried in your notifications :) )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vue 3 Support
3 participants