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

disabling negative inputs #22

Closed

Conversation

Tapesh-1308
Copy link
Contributor

Since we don't need negative input, there is no need to enable users to enter them. Therefore, I have disabled negative input by setting the minimum number as 0.

@sk66641
Copy link
Owner

sk66641 commented May 27, 2024

Thank you @Tapesh-1308 for your contribution! but could you please create an issue first describing the problem or feature this PR addresses? Once the issue is created, we can link it to this PR and proceed further.

@Tapesh-1308
Copy link
Contributor Author

@sk66641 I have created the issue. Now it can be linked with PR.

Issue number: #28

@sk66641
Copy link
Owner

sk66641 commented May 28, 2024

Ok, but we can still enter a negative input using keyboard i think, i tried it just now

@Tapesh-1308
Copy link
Contributor Author

@sk66641 Yes, you are right. Today, I've realized that the min/max attribute in HTML can be easily bypassed by inputting values from the keyboard. This means that JavaScript validation is still necessary, so my pull request doesn't make sense anymore. I'm going to close the pull request. I'm really sorry for the time I've taken from you. However, this doesn't discourage me. I will continue to contribute to the project. Thank you

@sk66641
Copy link
Owner

sk66641 commented May 29, 2024

@Tapesh-1308 No worries at all! Closing a pull request when you realize it’s not necessary or relevant is a responsible approach. Keep up the good work! 😊

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

Successfully merging this pull request may close these issues.

2 participants