fix(browser): remove global.window for node.js (#233)

fix(browser): remove global.window for node.js Close #232
naver · Oct 15, 2018 · b9ae32a · b9ae32a
1 parent f75590d
commit b9ae32a
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/packages/react-infinitegrid/src/consts.js b/packages/react-infinitegrid/src/consts.js
@@ -19,4 +19,4 @@ export const CHECK_ONLY_ERROR = 2;
 export const CONTAINER_CLASSNAME = "_eg-infinitegrid-container_";
 export const LAYOUT_ID = "__REACT_LAYOUT_DATA_ID__";
 export const DUMMY_POSITION = -999999;
-export const SUPPORT_COMPUTEDSTYLE = !!("getComputedStyle" in window);
+export const SUPPORT_COMPUTEDSTYLE = typeof window === "undefined" ? false : !!("getComputedStyle" in window);
diff --git a/src/Watcher.js b/src/Watcher.js
@@ -1,6 +1,9 @@
 import {
 	IS_IOS,
 } from "./consts";
+import {
+	window,
+} from "./browser";
 import {
 	addEvent,
 	removeEvent,

diff --git a/src/browser.js b/src/browser.js
@@ -1,14 +1,18 @@
 /* eslint-disable no-new-func, no-nested-ternary */
 
+let win;
+
 if (typeof window === "undefined") {
-	global.window = {
+	// window is undefined in node.js
+	win = {
 		document: {},
 		navigator: {
 			userAgent: "",
 		},
 	};
+} else {
+	win = window;
 }
-const win = window;
 /* eslint-enable no-new-func, no-nested-ternary */
 
 export {win as window};