Fix for WindowProvider utilities always returning window/document/und…

…efined (microsoft#31968)
spmonahan · Jul 17, 2024 · dd5120e · dd5120e
1 parent 8f32934
commit dd5120e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/change/@fluentui-react-cc5ae7ec-1f14-415d-915b-88de6d5cb7f0.json b/change/@fluentui-react-cc5ae7ec-1f14-415d-915b-88de6d5cb7f0.json
@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "Fix WindowProvider utilities always returning window/document/undefined",
+  "packageName": "@fluentui/react",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}
diff --git a/packages/react/src/utilities/dom.ts b/packages/react/src/utilities/dom.ts
@@ -17,7 +17,7 @@ import { useDocument, useWindow, WindowProviderProps } from '@fluentui/react-win
  */
 export const useDocumentEx = () => {
   // eslint-disable-next-line no-restricted-globals
-  return useDocument() ?? typeof document !== 'undefined' ? document : undefined;
+  return useDocument() ?? (typeof document !== 'undefined' ? document : undefined);
 };
 
 /**
@@ -27,7 +27,7 @@ export const useDocumentEx = () => {
  */
 export const useWindowEx = () => {
   // eslint-disable-next-line no-restricted-globals
-  return useWindow() ?? typeof window !== 'undefined' ? window : undefined;
+  return useWindow() ?? (typeof window !== 'undefined' ? window : undefined);
 };
 
 /**
@@ -39,7 +39,7 @@ export const useWindowEx = () => {
  */
 export const getDocumentEx = (ctx: Pick<WindowProviderProps, 'window'> | undefined) => {
   // eslint-disable-next-line no-restricted-globals
-  return ctx?.window?.document ?? typeof document !== 'undefined' ? document : undefined;
+  return ctx?.window?.document ?? (typeof document !== 'undefined' ? document : undefined);
 };
 
 /**
@@ -51,5 +51,5 @@ export const getDocumentEx = (ctx: Pick<WindowProviderProps, 'window'> | undefin
  */
 export const getWindowEx = (ctx: Pick<WindowProviderProps, 'window'> | undefined) => {
   // eslint-disable-next-line no-restricted-globals
-  return ctx?.window ?? typeof window !== 'undefined' ? window : undefined;
+  return ctx?.window ?? (typeof window !== 'undefined' ? window : undefined);
 };