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

Fixed issue with pasting text generates a javascript error #110

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

Conversation

dnetherton
Copy link

I hit a bug, where pasting content in masked input, generates a javascript error.

I havent looked to close, but i suspect, when the component was converted to class format (from createClass()), that the binding for the function _updateInputSelection was missed.

Also, not sure if there are other functions that also need binding.

Regards
Dean Netherton

@dwbruhn
Copy link
Contributor

dwbruhn commented Sep 28, 2017

👍

dwbruhn added a commit to dwbruhn/react-maskedinput that referenced this pull request Sep 28, 2017
Side effects:
  1) Delete behavior now allows removal of more than one character
  2) Cut now shifts remaining characters left instead of leaving "hole"

Also fix binding of this._updateInputSelection (cf. insin#110)
iamdustan pushed a commit that referenced this pull request Oct 20, 2017
Side effects:
  1) Delete behavior now allows removal of more than one character
  2) Cut now shifts remaining characters left instead of leaving "hole"

Also fix binding of this._updateInputSelection (cf. #110)
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

Successfully merging this pull request may close these issues.

2 participants