bypass bundler import for loading inert polyfill (#2145)

iTwin · Jul 16, 2024 · 774e584 · 774e584
1 parent e469f6a
commit 774e584
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/.changeset/few-candles-impress.md b/.changeset/few-candles-impress.md
@@ -0,0 +1,5 @@
+---
+'@itwin/itwinui-react': patch
+---
+
+Fixed an issue where the `Overlay` component was causing bundler warnings about a non-analyzable expression used in a dependency.
diff --git a/packages/itwinui-react/src/core/Overlay/Overlay.tsx b/packages/itwinui-react/src/core/Overlay/Overlay.tsx
@@ -3,7 +3,7 @@
  * See LICENSE.md in the project root for license terms and full copyright notice.
  *--------------------------------------------------------------------------------------------*/
 import * as React from 'react';
-import { Box, polymorphic } from '../../utils/index.js';
+import { Box, isUnitTest, polymorphic } from '../../utils/index.js';
 import type { PolymorphicForwardRefComponent } from '../../utils/index.js';
 
 type OverlayComponentProps = {
@@ -112,8 +112,12 @@ const useInertPolyfill = () => {
 
   React.useEffect(() => {
     (async () => {
-      if (!HTMLElement.prototype.hasOwnProperty('inert') && !loaded.current) {
-        await import(/* webpackIgnore: true */ /* @vite-ignore */ modulePath);
+      if (
+        !HTMLElement.prototype.hasOwnProperty('inert') &&
+        !loaded.current &&
+        !isUnitTest
+      ) {
+        await new Function('url', 'return import(url)')(modulePath);
         loaded.current = true;
       }
     })();