document --> WINDOW.document

Author: billyvg

packages/feedback/src/integration.ts

@@ -1,3 +1,4 @@
+import { WINDOW } from '@sentry/browser';
 import type { Integration } from '@sentry/types';
 import { isBrowser } from '@sentry/utils';
 import { logger } from '@sentry/utils';
@@ -46,6 +47,8 @@ function isBrowser(): boolean {
 type FeedbackConfiguration = Partial<FeedbackConfigurationWithDefaults>;
 >>>>>>> 5fa9a4abb (ref: extract widget creation to function, allow handling of multiple widgets)
 
+const doc = WINDOW.document;
+
 /**
  * Feedback integration. When added as an integration to the SDK, it will
  * inject a button in the bottom-right corner of the window that opens a
@@ -184,8 +187,7 @@ export class Feedback implements Integration {
       if (this._host) {
         this.remove();
       }
-      // eslint-disable-next-line no-restricted-globals
-      const existingFeedback = document.querySelector(`#${this.options.id}`);
+      const existingFeedback = doc.querySelector(`#${this.options.id}`);
       if (existingFeedback) {
         existingFeedback.remove();
       }
@@ -200,7 +202,6 @@ export class Feedback implements Integration {
 
       this._widget = this._createWidget(this.options);
     } catch (err) {
-      // TODO: error handling
       logger.error(err);
     }
   }
@@ -214,8 +215,7 @@ export class Feedback implements Integration {
 
       return this._ensureShadowHost<Widget | null>(options, ([shadow]) => {
         const targetEl =
-          // eslint-disable-next-line no-restricted-globals
-          typeof el === 'string' ? document.querySelector(el) : typeof el.addEventListener === 'function' ? el : null;
+          typeof el === 'string' ? doc.querySelector(el) : typeof el.addEventListener === 'function' ? el : null;
 
         if (!targetEl) {
           logger.error('[Feedback] Unable to attach to target element');
@@ -294,8 +294,7 @@ export class Feedback implements Integration {
       const widget = createWidget({ shadow, options });
 
       if (!this._hasInsertedActorStyles && widget.actor) {
-        // eslint-disable-next-line no-restricted-globals
-        shadow.appendChild(createActorStyles(document));
+        shadow.appendChild(createActorStyles(doc));
         this._hasInsertedActorStyles = true;
       }
 
@@ -333,8 +332,7 @@ export class Feedback implements Integration {
     const result = cb([this._shadow, this._host]);
 
     if (needsAppendHost) {
-      // eslint-disable-next-line no-restricted-globals
-      document.body.appendChild(this._host);
+      doc.body.appendChild(this._host);
     }
 
     return result;

packages/feedback/src/widget/Icon.ts

@@ -1,3 +1,4 @@
+import { WINDOW } from '@sentry/browser';
 import { setAttributesNS } from '../util/setAttributesNS';
 
 const SIZE = 20;
@@ -12,8 +13,7 @@ interface IconReturn {
  */
 export function Icon(): IconReturn {
   const cENS = <K extends keyof SVGElementTagNameMap>(tagName: K): SVGElementTagNameMap[K] =>
-    // eslint-disable-next-line no-restricted-globals
-    document.createElementNS(XMLNS, tagName);
+    WINDOW.document.createElementNS(XMLNS, tagName);
   const svg = setAttributesNS(cENS('svg'), {
     class: 'feedback-icon',
     width: `${SIZE}`,

packages/feedback/src/widget/SuccessIcon.ts

@@ -1,3 +1,4 @@
+import { WINDOW } from '@sentry/browser';
 import { setAttributesNS } from '../util/setAttributesNS';
 
 const WIDTH = 16;
@@ -13,8 +14,7 @@ interface IconReturn {
  */
 export function SuccessIcon(): IconReturn {
   const cENS = <K extends keyof SVGElementTagNameMap>(tagName: K): SVGElementTagNameMap[K] =>
-    // eslint-disable-next-line no-restricted-globals
-    document.createElementNS(XMLNS, tagName);
+    WINDOW.document.createElementNS(XMLNS, tagName);
   const svg = setAttributesNS(cENS('svg'), {
     class: 'success-icon',
     width: `${WIDTH}`,

packages/feedback/src/widget/createShadowHost.ts

@@ -1,3 +1,4 @@
+import { WINDOW } from '@sentry/browser';
 import { logger } from '@sentry/utils';
 
 import type { FeedbackConfigurationWithDefaults } from '../types';
@@ -7,12 +8,13 @@ import { createMainStyles } from './Main.css';
 interface CreateShadowHostParams {
   options: FeedbackConfigurationWithDefaults;
 }
+
 /**
- *
+ * Creates shadow host
  */
 export function createShadowHost({ options }: CreateShadowHostParams): [shadow: ShadowRoot, host: HTMLDivElement] {
-  // eslint-disable-next-line no-restricted-globals
-  const doc = document;
+  const doc = WINDOW.document;
+
   if (!doc.head.attachShadow) {
     // Shadow DOM not supported
     logger.warn('[Feedback] Browser does not support shadow DOM API');

packages/feedback/src/widget/util/createElement.ts

@@ -1,3 +1,5 @@
+import { WINDOW } from '@sentry/browser';
+
 /**
  * Helper function to create an element. Could be used as a JSX factory
  * (i.e. React-like syntax).
@@ -7,8 +9,8 @@ export function createElement<K extends keyof HTMLElementTagNameMap>(
   attributes: { [key: string]: string | boolean | EventListenerOrEventListenerObject } | null,
   ...children: any
 ): HTMLElementTagNameMap[K] {
-  // eslint-disable-next-line no-restricted-globals
-  const element = document.createElement(tagName);
+  const doc = WINDOW.document;
+  const element = doc.createElement(tagName);
 
   if (attributes) {
     Object.entries(attributes).forEach(([attribute, attributeValue]) => {
@@ -32,6 +34,7 @@ export function createElement<K extends keyof HTMLElementTagNameMap>(
 }
 
 function appendChild(parent: Node, child: any): void {
+  const doc = WINDOW.document;
   if (typeof child === 'undefined' || child === null) {
     return;
   }
@@ -43,12 +46,10 @@ function appendChild(parent: Node, child: any): void {
   } else if (child === false) {
     // do nothing if child evaluated to false
   } else if (typeof child === 'string') {
-    // eslint-disable-next-line no-restricted-globals
-    parent.appendChild(document.createTextNode(child));
+    parent.appendChild(doc.createTextNode(child));
   } else if (child instanceof Node) {
     parent.appendChild(child);
   } else {
-    // eslint-disable-next-line no-restricted-globals
-    parent.appendChild(document.createTextNode(String(child)));
+    parent.appendChild(doc.createTextNode(String(child)));
   }
 }