Merge pull request #636 from Kreijstal/globalThis-not-in-browser

check for globalThis.
ruilisi · Nov 25, 2024 · 2629e8b · 2629e8b
2 parents 9b67774 + 9164f1d
commit 2629e8b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/core/src/context.ts b/packages/core/src/context.ts
@@ -227,7 +227,7 @@ export function defaultContext(refs: RefValues): Context {
     defaultrowNum: 84,
     addDefaultRows: 50,
     fullscreenmode: true,
-    devicePixelRatio: (globalThis || window).devicePixelRatio,
+    devicePixelRatio: (typeof globalThis !== 'undefined' ? globalThis : typeof window !== 'undefined' ? window : {}).devicePixelRatio,
 
     contextMenu: {},
     sheetTabContextMenu: {},

diff --git a/packages/react/src/components/Sheet/index.tsx b/packages/react/src/components/Sheet/index.tsx
@@ -30,7 +30,7 @@ const Sheet: React.FC<Props> = ({ sheet }) => {
       if (!data) return;
       setContext((draftCtx) => {
         if (settings.devicePixelRatio === 0) {
-          draftCtx.devicePixelRatio = (globalThis || window).devicePixelRatio;
+          draftCtx.devicePixelRatio = (typeof globalThis !== 'undefined' ? globalThis : typeof window !== 'undefined' ? window : {}).devicePixelRatio;
         }
         updateContextWithSheetData(draftCtx, data);
         updateContextWithCanvas(