From: Ben Harris <bjh21@bjh21.me.uk>
Date: Sat, 4 Oct 2025 21:13:04 +0000 (+0100)
Subject: webstead: mark script as async
X-Git-Tag: bedstead-3.261~52
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~bjharris/git?a=commitdiff_plain;h=e6c1e44db08b438c40d1a14c8c98e1cc2f073583;p=bedstead.git

webstead: mark script as async

Apparently Chromium can't cope with "defer" scripts (which modules are
by default) in XHTML documents.  But "async" scripts work properly,
so now we use one of those and then wait for DOMContentLoaded as
necessary.
---

diff --git a/webstead.xhtml b/webstead.xhtml
index 61a261e..c3bb0c1 100644
--- a/webstead.xhtml
+++ b/webstead.xhtml
@@ -38,7 +38,10 @@
           background: currentColor;
       }
     </style>
-    <script type="module"><![CDATA[
+    <!-- Script is async rather than deferred because Chromium can't
+         handle deferred scripts in XHTML.
+         https://issues.chromium.org/issues/40518469 -->
+    <script type="module" async="async"><![CDATA[
     import Bedstead from './bedstead.js';
     function update_glyph() {
         var args = [];
@@ -71,10 +74,19 @@
             }
         });
     }
-    for (var e of document.getElementsByTagName('input')) {
-        e.onchange = update_glyph;
+    function init() {
+        for (var e of document.getElementsByTagName('input')) {
+            e.onchange = update_glyph;
+        }
+        update_glyph();
+    }
+    // This script is loaded asynchronously, so make sure the DOM is
+    // loaded before touching it.
+    if (document.readyState === "loading") {
+        document.addEventListener("DOMContentLoaded", init);
+    } else {
+        init();
     }
-    update_glyph();
 ]]>
     </script>
   </head>