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

onMaskComplete event #63

Closed
wants to merge 1 commit into from
Closed

onMaskComplete event #63

wants to merge 1 commit into from

Conversation

robinschuere
Copy link

@robinschuere robinschuere commented Oct 14, 2016

We have need for a specific behaviour for sending a component event whenever the mask length is filled in. By using the onChange event, the caret would always go to the end. This behaviour was counterproductive.
I added the onMaskComplete event for when this rule applies. => the event get's fired whenever the inputs length is equal to the mask length.

@robinschuere
Copy link
Author

@sanniassin, can you have a look at this PR? Thx in advance

@rodrigobacelli
Copy link

@sanniassin this PR will be merged to the project sometime in this life?

@sanniassin
Copy link
Owner

@robinschuere @rodrigobacelli Sorry for delay. I won't add any features which can be easily implemented in higher order component to keep this project as simple as possible. That's why i won't merge this PR.

@EgoPingvina
Copy link

the component that provides the functionality of masked input does not give a callback on filling in this very mask. Leaving the need to implement this basic function by the user every time is a decision above all praise, facepalm.

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.

4 participants