FIX: Delay entwine onmatch until DOMContentLoaded (fixes silverstripe…

…#1469)
kinglozzer · Mar 16, 2023 · bef5379 · bef5379
1 parent 8740b5e
commit bef5379
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 6 deletions.
diff --git a/client/dist/js/bundle.js b/client/dist/js/bundle.js
diff --git a/client/dist/js/vendor.js b/client/dist/js/vendor.js
diff --git a/thirdparty/jquery-entwine/jquery.entwine.js b/thirdparty/jquery-entwine/jquery.entwine.js
@@ -1382,17 +1382,18 @@ Sizzle is good for finding elements for a selector, but not so good for telling
 		$(document).triggerHandler(event);
 	});
 
-})(jQuery);;
+})(jQuery);
 
 
 /* src/domevents/jquery.entwine.domevents.maybechanged.js */
 
 (function($){
 
-	/** What to call to run a function 'soon'. Normally setTimeout, but for syncronous mode we override so soon === now */
-	var runSoon = window.setTimeout;
+  // Helper function to run a function "soon". This is effectively using setTimeout, but ensuring it doesn't run
+  // before DOMContentLoaded (via jQuery.ready())
+  var runSoon = (fn, delay) => $(_ => window.setTimeout(fn, delay));
 
-	/** The timer handle for the asyncronous matching call */
+	/** The timer handle for the asynchronous matching call */
 	var ChangeDetails = Base.extend({
 
 		init: function() {