From f74840631ba67013503a4c2f62f0856773ae37fc Mon Sep 17 00:00:00 2001
From: Hypnosphi <Filipp.Riabchun@jetbrains.com>
Date: Mon, 9 Apr 2018 03:00:11 +0300
Subject: [PATCH] Remove cWRP usage in ModalPortal

---
 src/components/ModalPortal.js | 29 ++++++++++++-----------------
 1 file changed, 12 insertions(+), 17 deletions(-)

diff --git a/src/components/ModalPortal.js b/src/components/ModalPortal.js
index 5220003d..4ad62d2b 100644
--- a/src/components/ModalPortal.js
+++ b/src/components/ModalPortal.js
@@ -69,23 +69,21 @@ export default class ModalPortal extends Component {
   }
 
   componentDidMount() {
-    // Focus needs to be set when mounting and already open
     if (this.props.isOpen) {
-      this.setFocusAfterRender(true);
       this.open();
     }
   }
 
-  componentWillReceiveProps(newProps) {
+  componentDidUpdate(prevProps, prevState) {
     if (process.env.NODE_ENV !== "production") {
-      if (newProps.bodyOpenClassName !== this.props.bodyOpenClassName) {
+      if (prevProps.bodyOpenClassName !== this.props.bodyOpenClassName) {
         // eslint-disable-next-line no-console
         console.warn(
           'React-Modal: "bodyOpenClassName" prop has been modified. ' +
             "This may cause unexpected behavior when multiple modals are open."
         );
       }
-      if (newProps.htmlOpenClassName !== this.props.htmlOpenClassName) {
+      if (prevProps.htmlOpenClassName !== this.props.htmlOpenClassName) {
         // eslint-disable-next-line no-console
         console.warn(
           'React-Modal: "htmlOpenClassName" prop has been modified. ' +
@@ -93,19 +91,20 @@ export default class ModalPortal extends Component {
         );
       }
     }
-    // Focus only needs to be set once when the modal is being opened
-    if (!this.props.isOpen && newProps.isOpen) {
-      this.setFocusAfterRender(true);
+
+    if (this.props.isOpen && !prevProps.isOpen) {
       this.open();
-    } else if (this.props.isOpen && !newProps.isOpen) {
+    } else if (!this.props.isOpen && prevProps.isOpen) {
       this.close();
     }
-  }
 
-  componentDidUpdate() {
-    if (this.focusAfterRender) {
+    // Focus only needs to be set once when the modal is being opened
+    if (
+      this.props.shouldFocusAfterRender &&
+      this.state.isOpen &&
+      !prevState.isOpen
+    ) {
       this.focusContent();
-      this.setFocusAfterRender(false);
     }
   }
 
@@ -114,10 +113,6 @@ export default class ModalPortal extends Component {
     clearTimeout(this.closeTimer);
   }
 
-  setFocusAfterRender = focus => {
-    this.focusAfterRender = this.props.shouldFocusAfterRender && focus;
-  };
-
   setOverlayRef = overlay => {
     this.overlay = overlay;
     this.props.overlayRef && this.props.overlayRef(overlay);