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

allow delete keypresses in v-input #10264

Merged
merged 1 commit into from Dec 3, 2021
Merged

allow delete keypresses in v-input #10264

merged 1 commit into from Dec 3, 2021

Conversation

azrikahar
Copy link
Contributor

Since there's backspace, allowing delete should feel more natural.

Also noticed that props.dbSafe actually uses key.startsWith('arrow') to check for arrow keys, so I removed them from the systemKeys array and added the same logic for props.slug.

const isAllowed = dbSafeCharacters.includes(key) || systemKeys.includes(key) || key.startsWith('arrow');

@rijkvanzanten rijkvanzanten added this to the v9.1.3 milestone Dec 3, 2021
@rijkvanzanten rijkvanzanten self-assigned this Dec 3, 2021
@rijkvanzanten rijkvanzanten merged commit 2dc4908 into main Dec 3, 2021
@rijkvanzanten rijkvanzanten deleted the input-delete-key branch December 3, 2021 16:11
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants