Skip to content
This repository has been archived by the owner on Sep 4, 2024. It is now read-only.

Press And hold key overrides mask limit #16

Closed
fimdomeio opened this issue Jan 12, 2017 · 7 comments
Closed

Press And hold key overrides mask limit #16

fimdomeio opened this issue Jan 12, 2017 · 7 comments

Comments

@fimdomeio
Copy link

for a mask that is something like v-mask="'##h##'" if I press the key 1 continuously the input will be 11h111.
Don't know if it matters but I have a fast repeating rate in my keyboard.

@probil
Copy link
Owner

probil commented Jan 12, 2017

I will check. Thanks for your feedback/response

@probil
Copy link
Owner

probil commented Feb 13, 2017

Please, provide some info about your system. At least, library version, vue version, browser and system

@probil
Copy link
Owner

probil commented Feb 13, 2017

Can't reproduce on my machine

@fimdomeio
Copy link
Author

"v-mask": "^1.0.2",
"vue": "^2.0.5",
Mac, Chrome 56. Just noticed v-mask is not working at all on Firefox. This is probably a problem on my side, I'll investigate further.

From testing I realized the problem is actually a bit different from my initial explanation. Chrome is returning one more character than expected in v-model. Inspecting further it starts to look like I shouldn't be using v-model at all but somehow hooking up to data-previous-value.

So I would say this is probably not a bug, but I would suggest adding to the documentation an example of outputing the value..

I'll look more deeply into this in the next few days.

@fimdomeio
Copy link
Author

A video of my issue.
https://fimdomeio.com/p/booknatura/v-mask-trouble.m4v

@probil
Copy link
Owner

probil commented Feb 14, 2017

Aha! I got it. Displayed value and mask value is different. Thanks, @fimdomeio

probil pushed a commit that referenced this issue Mar 6, 2017
- Fixed issue with excessive chars inside model data
- Simplified directive by removing custom event listeners
probil pushed a commit that referenced this issue Apr 2, 2017
- Fixed issue with excessive chars inside model data
- Simplified directive by removing custom event listeners
probil pushed a commit that referenced this issue Apr 2, 2017
- Fixed issue with excessive chars inside model data
- Simplified directive by removing custom event listeners
@probil
Copy link
Owner

probil commented Apr 3, 2017

Please, check the v1.1.0 and up. Should be fixed

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

No branches or pull requests

2 participants