FIX: Delay entwine onmatch until DOMContentLoaded (fixes silverstripe…

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

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
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 to run a function "soon". In "synchronous" mode this is overridden so soon === now, but by default this uses
+  // setTimeout wrapped in $() (shorthand for $(document).ready()) to ensure it never runs before DOMContentLoaded
+  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() {