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

Replacing "keypress" event binding to "keyup" as "keypress" is #78

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

Conversation

tpresthus
Copy link

Replacing "keypress" event binding to "keyup" as "keypress" isn't covered by any official specification and doesn't work on all browsers, like the Android default browser as observed in #71

deprecated and doesn't work on all browsers, like the Android default
browser.
@tpresthus
Copy link
Author

Looks like keyup has some side-effects that I didn't notice. Investigating further.

@bendrucker
Copy link

Give angular-credit-cards 2.3 a shot. I've added formatting that does a reasonably good job of getting the cursor position right without breaking out of Angular's native input handling events. It's quite consistent and simple for now which I think is best until I can really devote the time to aggressive cross-browser testing of a vanilla JS approach.

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

Successfully merging this pull request may close these issues.

2 participants