jQuery-iMask icon indicating copy to clipboard operation
jQuery-iMask copied to clipboard

Sanity issue

Open stefanneculai opened this issue 13 years ago • 1 comments

Hey,

I think that this.sanityTest( this.domNode.value ); should be done on keyPress too, because in the current way it's not possible to limit the value of a number.

return Math.min( 1000000, Math.max(0, parseFloat( val.replace(/[^.\d]/g, ''))) will allow a number higher than 1000000

stefanneculai avatar Oct 22 '12 19:10 stefanneculai

I second this one too.

cosmoKenney avatar Apr 12 '13 21:04 cosmoKenney