From 757f55e8050bbee197d62d4f45b15cd85be918ed Mon Sep 17 00:00:00 2001
From: Mitchell R <github@mrincworld.com>
Date: Sun, 2 Feb 2025 17:04:29 +0200
Subject: [PATCH] Fixing #3165

Fixes case where htmx is loaded async and DOMContentLoaded is never fired.
---
 src/htmx.js | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/src/htmx.js b/src/htmx.js
index 424da472c..ee1f3a75b 100644
--- a/src/htmx.js
+++ b/src/htmx.js
@@ -5012,9 +5012,20 @@ var htmx = (function() {
   // Initialization
   //= ===================================================================
   var isReady = false
-  getDocument().addEventListener('DOMContentLoaded', function() {
+  var readyFunctions = []
+  /*
+   * When the HTMX lib is loaded async, the DOMContentLoaded event never fires, so the
+   * ready functions never trigger. (https://github.com/bigskysoftware/htmx/issues/3165)
+   * To make sure this never happens, we will listen to both DOMContentLoaded and window.onload
+   */
+  function triggerReadyFunctions() {
+    if (!readyFunctions) { return }
     isReady = true
-  })
+    readyFunctions.forEach(x => x())
+    readyFunctions = undefined
+  }
+  getDocument().addEventListener('DOMContentLoaded', triggerReadyFunctions)
+  window.onload = triggerReadyFunctions
 
   /**
    * Execute a function now if DOMContentLoaded has fired, otherwise listen for it.
@@ -5028,8 +5039,9 @@ var htmx = (function() {
     // some means other than the initial page load.
     if (isReady || getDocument().readyState === 'complete') {
       fn()
+      triggerReadyFunctions()
     } else {
-      getDocument().addEventListener('DOMContentLoaded', fn)
+      readyFunctions.push(fn)
     }
   }