8000 `delete` key modifier includes backspace? · Issue #3557 · vuejs/vue · GitHub
[go: up one dir, main page]

Skip to content

delete key modifier includes backspace? #3557

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

Closed
simshaun opened this issue Aug 30, 2016 · 3 comments
Closed

delete key modifier includes backspace? #3557

simshaun opened this issue Aug 30, 2016 · 3 comments

Comments

@simshaun
Copy link

This is a bit confusing to me. I expected my event to only be triggered by the Delete key, but it's also being triggered by the Backspace key.

I see that keyCodes.delete includes the keycodes for both Backspace & Delete.

Would it not make more sense for keyCodes.delete to just be the Delete key? A new keyCode could be added for the Backspace key.

@thecrypticace
Copy link

See #2211

@yyx990803
Copy link
Member

This is an intentional decision, and probably will remain so because changing it would be breaking. It is pretty straightforward to workaround it by using explicit keyCodes.

@simshaun
Copy link
Author
simshaun commented Aug 30, 2016

I would never have guessed the current functionality because I've never used a Mac keyboard, so the problem described in #2211 is a non-issue for me and the solution only created confusion on my part.

I feel like the docs should at least state that "delete" handles both the Delete key & Backspace key (if user's keyboard has it). I created a PR if you're interested. vuejs/v2.vuejs.org#385

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

No branches or pull requests

3 participants
0