fix review comments

Author: samthor

build/modern-loader.js

@@ -98,13 +98,13 @@ module.exports = async (id, content, onwarn=null) => {
     // so we should only end up with a single result (checked below).
     preserveModules: true,
     onwarn,
+    treeshake: false,  // we don't want any code thrown away in dev
   });
 
   const out = await bundle.generate({
     name: id,
     format: 'es',
     sourcemap: true,
-    //treeshake: false,  // we want to cache the results
   });
 
   if (out.output.length !== 1) {

static/scenes/modvil/elements/modvil-tracker.js

@@ -38,7 +38,9 @@ common.preload.images(
 const focusTimeoutDelay = 20 * 1000;  // refocus on Santa after this much inactivity
 
 
-// This 
+// The jitter ratio is stored for the current browser window, so if users reload, they'll see Santa
+// "jittered" by the same amount. This lets us randomly skew Santa through the route via Firebase,
+// in case Maps is having a bad time.
 if (!localStorage['routeJitter']) {
   localStorage['routeJitter'] = (Math.random() * 2) - 1;  // between [-1,+1]
 }