revert trap focus changes

mui · Jun 18, 2019 · 6ebe7f6 · 6ebe7f6
1 parent 69420df
commit 6ebe7f6
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/packages/material-ui/src/Modal/TrapFocus.js b/packages/material-ui/src/Modal/TrapFocus.js
@@ -26,10 +26,6 @@ function TrapFocus(props) {
   const sentinelStart = React.useRef(null);
   const sentinelEnd = React.useRef(null);
   const nodeToRestore = React.useRef();
-  const isEnabledRef = React.useRef();
-  useEnhancedEffect(() => {
-    isEnabledRef.current = isEnabled;
-  }, []);
 
   const rootRef = React.useRef(null);
   // can be removed once we drop support for non ref forwarding class components
@@ -74,7 +70,7 @@ function TrapFocus(props) {
     }
 
     const contain = () => {
-      if (disableEnforceFocus || !isEnabledRef.current() || ignoreNextEnforceFocus.current) {
+      if (disableEnforceFocus || !isEnabled() || ignoreNextEnforceFocus.current) {
         ignoreNextEnforceFocus.current = false;
         return;
       }
@@ -86,7 +82,7 @@ function TrapFocus(props) {
 
     const loopFocus = event => {
       // 9 = Tab
-      if (disableEnforceFocus || !isEnabledRef.current() || event.keyCode !== 9) {
+      if (disableEnforceFocus || !isEnabled() || event.keyCode !== 9) {
         return;
       }
 
@@ -123,7 +119,7 @@ function TrapFocus(props) {
         nodeToRestore.current = null;
       }
     };
-  }, [disableAutoFocus, disableEnforceFocus, disableRestoreFocus, open]);
+  }, [disableAutoFocus, disableEnforceFocus, disableRestoreFocus, isEnabled, open]);
 
   return (
     <React.Fragment>