Skip to content

Decimal dynamic and add static masked method#138

Open
micronull wants to merge 5 commits intoplentz:masterfrom
micronull:master
Open

Decimal dynamic and add static masked method#138
micronull wants to merge 5 commits intoplentz:masterfrom
micronull:master

Conversation

@micronull
Copy link

  1. If decimalAfterDot = true, fractional number when the user presses a key and a dot or a comma, or integral number without a fraction.
  2. Add static method masked. When you need only apply the mask to the tag. Without events.

1. If decimalAfterDot = true, fractional number when the user presses a key and a dot or a comma, or integral number without a fraction.
2. Add static method masked. When you need only apply the mask to the tag. Without events.
@micronull
Copy link
Author

See example http://jsfiddle.net/a0tekg6z/

@pilap82
Copy link

pilap82 commented Aug 29, 2014

My users will love it when it's merged! Thank you!

@pilap82
Copy link

pilap82 commented Aug 29, 2014

FYI, the fiddle doesn't work, my browser refuses to load the script (Chrome, OSX).
However, if I copy / paste the content of the external script at the beginning of the Javascript pane of JSfiddle, it works fine.

@micronull
Copy link
Author

Cool. I'm changed the function to the class, so now it will be easier to add functionality.

Correct example for chrome. I used firefox on the old example.
http://jsfiddle.net/a0tekg6z/1/

@plentz
Copy link
Owner

plentz commented Feb 28, 2015

@De-Luxis sorry for taking so long for reply! A lot of users already asked to allow something like your pull request, but this is missing one point: formatting the number on the blur(example, if the users set the precision to 2, on blur, 1000 must be 1000,00, even if the user only typed 1000, since we need to apply the mask that the user asked for). Another thing: you changed a lot of the indentation and it's pretty hard to know exactly which parts of the code you have changed. Can you update the PR keeping the current indentation? (like what appears when you look here https://github.com/plentz/jquery-maskmoney/pull/138/files?w=1)

Thanks in advance, you rock :)

@plentz plentz added the feature label Mar 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments