Apply React polyfill directly to globalThis

Shopify · Nov 14, 2024 · b4e67ce · b4e67ce
1 parent 4a2242e
commit b4e67ce
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 18 deletions.
diff --git a/.changeset/sharp-frogs-bathe.md b/.changeset/sharp-frogs-bathe.md
@@ -0,0 +1,5 @@
+---
+'@remote-dom/react': patch
+---
+
+Apply React polyfill directly to `globalThis`
diff --git a/packages/react/source/polyfill.ts b/packages/react/source/polyfill.ts
@@ -1,5 +1,3 @@
-import {window} from '@remote-dom/core/polyfill';
-
 class HTMLIFrameElement extends HTMLElement {}
 
 class CSSStyleDeclaration {
@@ -24,23 +22,8 @@ class CSSStyleDeclaration {
   }
 }
 
-// React ues the `location` property when printing help text in development.
-if (!window.location) {
-  Object.defineProperty(window, 'location', {
-    value: globalThis.location ?? {protocol: 'https:'},
-    configurable: true,
-  });
-}
-
-if (!window.navigator) {
-  Object.defineProperty(window, 'navigator', {
-    value: globalThis.navigator ?? {userAgent: ''},
-    configurable: true,
-  });
-}
-
 // React checks whether elements are Iframes on initialization.
-Object.defineProperty(window, 'HTMLIFrameElement', {
+Object.defineProperty(globalThis, 'HTMLIFrameElement', {
   value: HTMLIFrameElement,
   configurable: true,
 });
@@ -61,3 +44,5 @@ Object.defineProperty(Element.prototype, 'style', {
     this.style.cssText = String(cssText);
   },
 });
+
+export {};