Build the Webworker into an iife instead of an ES module for compatib…

…ility with older browsers (#69)
Doenet · Dec 22, 2023 · f58aee4 · f58aee4
1 parent a89f0de
commit f58aee4
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 4 deletions.
diff --git a/packages/doenetml-worker/vite.config.ts b/packages/doenetml-worker/vite.config.ts
@@ -9,8 +9,9 @@ export default defineConfig({
         sourcemap: true,
         lib: {
             entry: "src/index.ts",
-            fileName: "CoreWorker",
-            formats: ["es"],
+            fileName: (_format, _entryName) => "CoreWorker.js",
+            formats: ["iife"],
+            name: "doenetmlWorker"
         },
         rollupOptions: {
             // @ts-ignore

diff --git a/packages/doenetml/src/utils/activityUtils.js b/packages/doenetml/src/utils/activityUtils.js
@@ -684,7 +684,7 @@ export async function determineNumberOfActivityVariants(
 
 export function createCoreWorker() {
     return new Worker(doenetGlobalConfig.doenetWorkerUrl, {
-        type: "module",
+        type: "classic",
     });
 }
 

diff --git a/packages/doenetml/src/utils/activtyWebWorker.js b/packages/doenetml/src/utils/activtyWebWorker.js
@@ -1,4 +1,6 @@
 export function prerenderActivity({ cid, doenetId, flags }) {
+    // TODO - do we need to move this web worker to an iife as well?
+    // see - https://github.com/Doenet/DoenetML/pull/69
     let worker = new Worker("/utils/prerenderWorker.js", { type: "module" });
 
     // console.log(`Prerendering activity`, cid, doenetId, flags, worker);

diff --git a/packages/doenetml/src/utils/prerenderWorker.js b/packages/doenetml/src/utils/prerenderWorker.js
@@ -217,7 +217,7 @@ function calculateInitialRendererState({
     flags = {},
 }) {
     let coreWorker = new Worker(doenetGlobalConfig.doenetWorkerUrl, {
-        type: "module",
+        type: "classic",
     });
 
     coreWorker.postMessage({