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

maxLength settings and fix maxLength when used allowNegative and prefix #200

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

brunoguadagnini
Copy link

@brunoguadagnini brunoguadagnini commented Jun 15, 2016

fix:max maxLength value when used allowNegative and prefix
add: settings "maxLength" to limit the quantity of numeric characters

brunoguadagnini and others added 3 commits June 15, 2016 12:24
add settings "numeralMaxLength" (default: false) to limit the quantity of numeric characters

Ex:

<input type="text" id="demo" maxlength="9" />
$('#demo').maskMoney({prefix:'R$ ', thousands:'.', decimal:',', allowZero: true, numeralMaxLength: true});
//max value= R$ 9.999.999,99

$('#demo').maskMoney({prefix:'R$ ', thousands:'.', decimal:',', allowZero: true, numeralMaxLength: false});
//max value= R$ 999,99
@brunoguadagnini brunoguadagnini changed the title Add numeralMaxLength settings and fix maxLength when used allowNegative and prefix maxLength settings and fix maxLength when used allowNegative and prefix Jun 17, 2016
@gkatsanos
Copy link

@plentz can we merge this?

@aureliosaraiva
Copy link
Collaborator

@brunoguadagnini
Can you implement unit test for bug fix?

@aureliosaraiva
Copy link
Collaborator

@brunoguadagnini can you resolve this conflict for merged.

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

Successfully merging this pull request may close these issues.

3 participants