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

Corrected issue regarding browser validation message for float number inputs #11927

Merged
merged 1 commit into from
Dec 2, 2022

Conversation

EstFoisy
Copy link
Contributor

What it does

For the preference number input if a float should be accepted in that field there will be no more a failure message showing.
Closes #11851

How to test

  1. Go into file: preference: Open Setting
  2. Find a field that should accept a float value
  3. Enter a float value
  4. There should be no message appearing
  5. Try a field that should only take an integer
  6. Enter a float value
  7. The failure message regarding the value being incorrect should appear
Issue.11851.-.before.correction.mp4
Issue.11851.-.after.correction.mp4

Review checklist

Reminder for reviewers

message for float number inputs

For the preference number input if a float should be accepted in that
field there will be no more a failure message showing.
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

I confirmed that:

  • integer preferences (ex: files.maxConcurrentUploads) do not accept float values
  • number preferences that are integers will step by 1
  • number preferences that are floats will not display the default browser validation
  • the change aligns with a recent change in vscode (as they also had the bug)

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

Successfully merging this pull request may close these issues.

Incorrect browser validation message for number inputs
2 participants