remove focustrap from loaderoverlay

dequelabs · Nov 21, 2024 · aeaba08 · aeaba08
1 parent 51c9362
commit aeaba08
Showing 1 changed file with 16 additions and 25 deletions.
diff --git a/packages/react/src/components/LoaderOverlay/index.tsx b/packages/react/src/components/LoaderOverlay/index.tsx
@@ -1,9 +1,9 @@
 import React, { forwardRef, useEffect } from 'react';
-import FocusTrap from 'focus-trap-react';
 import classNames from 'classnames';
 import Loader from '../Loader';
 import AxeLoader from './axe-loader';
 import AriaIsolate from '../../utils/aria-isolate';
+import useFocusTrap from '../../utils/useFocusTrap';
 import useSharedRef from '../../utils/useSharedRef';
 
 export interface LoaderOverlayProps
@@ -48,33 +48,24 @@ const LoaderOverlay = forwardRef<HTMLDivElement, LoaderOverlayProps>(
       }
     }, []);
 
-    const Wrapper = focusTrap ? FocusTrap : React.Fragment;
-    const wrapperProps = focusTrap
-      ? {
-          focusTrapOptions: {
-            fallbackFocus: '.Loader__overlay'
-          }
-        }
-      : {};
+    useFocusTrap(overlayRef, {
+      disabled: !focusTrap
+    });
 
     return (
-      <Wrapper {...wrapperProps}>
-        <div
-          className={classNames('Loader__overlay', className)}
-          ref={overlayRef}
-          tabIndex={-1}
-          {...other}
-        >
-          <div className="Loader__overlay__loader">
-            <Loader />
-            <AxeLoader />
-          </div>
-          {label ? (
-            <span className="Loader__overlay__label">{label}</span>
-          ) : null}
-          {children}
+      <div
+        className={classNames('Loader__overlay', className)}
+        ref={overlayRef}
+        tabIndex={-1}
+        {...other}
+      >
+        <div className="Loader__overlay__loader">
+          <Loader />
+          <AxeLoader />
         </div>
-      </Wrapper>
+        {label ? <span className="Loader__overlay__label">{label}</span> : null}
+        {children}
+      </div>
     );
   }
 );