-
Notifications
You must be signed in to change notification settings - Fork 481
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
Bug : Values in Forms can be Negative #7272
Comments
@rithviknishad, there are other forms also where the minimum value is not defined. Should I also fix them in the same PR? |
Yup, go ahead! Update the issue description and title too |
Okay :) |
@Keshav-0907 re-request for review once the changes are done. |
@rithviknishad @Keshav-0907 can we have frontend validation like this rather than the browser validation |
Absolutely, @nihal467 ! Having frontend validation instead of relying solely on browser validation is a great idea. It provides a more seamless and user-friendly experience. |
@nihal467 @rithviknishad @Ashesh3 Sir I have updated the PR #7273 by implementing frontend validation instead of browser validation. |
Describe the bug
Currently in forms users are allowed to input negative values in places where it's should be not allowed.
To Reproduce
(one of the examples)
Expected behavior
The form should not allow for negative values, and the minimum value accepted should be 0.
Screenshots
Screen.Recording.2024-02-26.at.12.05.35.PM.mov
Desktop (please complete the following information):
Smartphone (please complete the following information):
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: