-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Incorrect empty check fails on 0 range value #3458
Comments
@c-dante Good catch, are you willing to provide a fix for the |
There's another issue where the value is undefined it flips the element from begin controlled/uncontrolled and throws a lot of warnings during development. Can likely patch that, too |
@heath-freenome let me know what else is needed #3465 not sure what your release process is re updating package semver |
Is there an issue related to this problem? If not, please create one with reproducible steps and then provide a PR if you feel up to it. |
react-jsonschema-form/packages/mui/src/RangeWidget/RangeWidget.tsx
Line 39 in 8be4ef2
For ranges that include
0
, this falsy check switches the value to the default instead of the correct value.The text was updated successfully, but these errors were encountered: