Don't forget we need to apply this fix: #15231 (comment)

When I try converting the getDerivedStateFromProps method like

-static getDerivedStateFromProps(nextProps, prevState) {
-  if (typeof prevState.exited === 'undefined') {
-    return {
-      exited: !nextProps.open,
-    };
-  }
-
-  if (nextProps.open) {
-    return {
-      exited: false,
-    };
-  }
-
-  return null;
-}

+if (typeof exited === 'undefined') {
+  setExited(!open);
+} else if (open) {
+  setExited(false);
+}

if (typeof exited === 'undefined') {
  setExited(!open);
}

Reference: https://github.com/mui-org/material-ui/pull/15208/files#r272822227

Don't forget we need to apply this fix: #15231 (comment)

🌮 💯

Don't forget we need to apply this fix: #15231 (comment)

🌮 💯

That fix means you no longer need getDerivedStateFromProps

That fix means you no longer need getDerivedStateFromProps

Yes I am looking into it 😄 😄 I didn't see your comment before I posted that long useless comment of mine 🏷 🤣

@joshwooding Can you kindly review, if you think this implementation is okay, should I move towards the tests?

@joshwooding Can you kindly review, if you think this implementation is okay, should I move towards the tests?

Looks good so far. You need to remove the first export though. Haha you already did :P

@material-ui/core: parsed: -0.38% 😍, gzip: -0.14% 😍

Generated by 🚫 dangerJS against 7a65308

@adeelibr Well done :)