-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Thousands separators allowed for numbers, but not min/max #379
Comments
Duplicate of #188 - should integrate Globalize to solve this properly. |
+1 to the above. This is a joke. Implement the common case (whichever international format is preferred for the library's authors) and give developers a way to extend/change it. |
In case anyone hits this and needs a quick, U.S.-but-customizable fix, I already had numeral.js involved and using it to preformat values was a pinch: https://gist.github.com/joeRinehart/0e8af523fdfc567cc670ddca2eaff137 |
While trying to validate a number field with min and max attributes, I noticed that there was no invalid number complaint for something like "9,000" but that adding commas would cause the value to incorrectly fail min/max validation.
The text was updated successfully, but these errors were encountered: