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

updated input type and fixed type-o #111

Closed
wants to merge 2 commits into from

Conversation

randallard
Copy link

@gbj
Copy link
Contributor

gbj commented Jun 21, 2024

This is a fairly common observation/PR (#100, leptos-rs/leptos#2171) but it is not actually correct -- i.e., some browsers (Chrome) enforce some of the requirements you want this way, but allow other things that still yield errors (in this case, 1.23, e, or 240823412384134123432423412341 will all be allowed by <input type="number"> but still generate errors). Other browsers (Firefox, Safari) allow any input.

I have added a note to clarify and hopefully avoid this confusion in the future.

@gbj gbj closed this Jun 21, 2024
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