fix(userLocales): guard against window on node enviroments

wopian · wopian · commit 96d590cf447f · 2020-07-14T02:35:15.000+01:00
CodeSandBox was defining window and global.
diff --git a/README.md b/README.md
@@ -24,7 +24,7 @@ This is a monorepo containing the following packages:
 
 | Package            | Package<br>Size | Node | Chrome | Firefox | Safari | Edge |
 | -----------------: | :-------------: | :--: | :----: | :-----: | :----: | :--: |
-| `preferred-locale` | ≤ 642 bytes     | 10+  | 69+    | 68+     | 12+    | 18+  |
+| `preferred-locale` | ~580 bytes      | 10+  | 69+    | 68+     | 12+    | 18+  |
 
 ## Why?
 
diff --git a/packages/preferred-locale/README.md b/packages/preferred-locale/README.md
@@ -24,7 +24,7 @@
 
 |            Package | Package<br>Size | Node | Chrome | Firefox | Safari | Edge |
 | -----------------: | :-------------: | :--: | :----: | :-----: | :----: | :--: |
-| `preferred-locale` |   ≤ 642 bytes   |  10+ |   69+  |   68+   |   12+  |  18+ |
+| `preferred-locale` | ~580 bytes      |  10+ |   69+  |   68+   |   12+  |  18+ |
 
 ## Why?
 
diff --git a/packages/preferred-locale/src/utils/userLocales/index.js b/packages/preferred-locale/src/utils/userLocales/index.js
@@ -5,7 +5,7 @@
  * @private
  */
 export const userLocales = fallback => {
-  if (global?.navigator?.languages) return navigator.languages.concat([ fallback ])
+  if (typeof window === 'object' && window?.navigator?.languages) return window.navigator.languages.concat([ fallback ])
   if (Intl && 'DateTimeFormat' in Intl) return [ Intl.DateTimeFormat().resolvedOptions().locale, fallback ]
   return [ fallback ]
 }

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`	`* @private`
`6`	`6`	`*/`
`7`	`7`	`export const userLocales = fallback => {`
`8`		`- if (global?.navigator?.languages) return navigator.languages.concat([ fallback ])`
	`8`	`+ if (typeof window === 'object' && window?.navigator?.languages) return window.navigator.languages.concat([ fallback ])`
`9`	`9`	`if (Intl && 'DateTimeFormat' in Intl) return [ Intl.DateTimeFormat().resolvedOptions().locale, fallback ]`
`10`	`10`	`return [ fallback ]`
`11`	`11`	`}`