[utils] Allow passing NaN as defaultValue to useControlled

mui · Mar 20, 2024 · a330197 · a330197
1 parent 9585d28
commit a330197
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/packages/mui-utils/src/useControlled/useControlled.js b/packages/mui-utils/src/useControlled/useControlled.js
@@ -29,7 +29,7 @@ export default function useControlled({ controlled, default: defaultProp, name,
     const { current: defaultValue } = React.useRef(defaultProp);
 
     React.useEffect(() => {
-      if (!isControlled && defaultValue !== defaultProp) {
+      if (!isControlled && !Object.is(defaultValue, defaultProp)) {
         console.error(
           [
             `MUI: A component is changing the default ${state} state of an uncontrolled ${name} after being initialized. ` +

diff --git a/packages/mui-utils/src/useControlled/useControlled.test.js b/packages/mui-utils/src/useControlled/useControlled.test.js
@@ -105,4 +105,10 @@ describe('useControlled', () => {
       setProps({ defaultValue: 1 });
     }).not.toErrorDev();
   });
+
+  it('should not raise a warning if setting NaN as the defaultValue when uncontrolled', () => {
+    expect(() => {
+      render(<TestComponent defaultValue={NaN}>{() => null}</TestComponent>);
+    }).not.toErrorDev();
+  });
 });