fix(Javascript): Allow themes to be loaded in Node.js

stencila · Feb 12, 2020 · 4f307e5 · 4f307e5
1 parent 3b50050
commit 4f307e5
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/shared/js/index.ts b/src/shared/js/index.ts
@@ -58,6 +58,10 @@ const onReadyHandler = (): void => {
 }
 
 export const load = (): void => {
+  // Do not do anything if not in the browser
+  // (e.g. when loading themes in Node.js)
+  if (typeof window === 'undefined') return
+
   if (document.readyState === 'loading') {
     document.addEventListener('DOMContentLoaded', onReadyHandler)
   } else {

diff --git a/src/themes/plos/index.ts b/src/themes/plos/index.ts
@@ -1,3 +1,11 @@
+import { load } from '../../shared/js'
+
+load()
+
+/*
+This is old code, currently commented out with the view to
+consolidating in shared.
+
 export const init = (): void => {
   const referenceListItemSel = '[itemprop="references"] > li'
   const datePublishedSel = '[itemprop="datePublished"]'
@@ -22,3 +30,4 @@ export const init = (): void => {
 }
 
 document.addEventListener('DOMContentLoaded', init)
+*/