Photon: remove wp-dom-ready dependency

...this pulls in wp-polyfill which is a lot of kbs for nothing in Photon’s case since it’s ES5.
Automattic · Oct 2, 2019 · ac57bae · ac57bae
1 parent 5321876
commit ac57bae
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/class.photon.php b/class.photon.php
@@ -1221,8 +1221,8 @@ public function action_wp_enqueue_scripts() {
 				'_inc/build/photon/photon.min.js',
 				'modules/photon/photon.js'
 			),
-			array( 'wp-dom-ready' ),
-			20190901,
+			array(),
+			20191001,
 			true
 		);
 	}

diff --git a/modules/photon/photon.js b/modules/photon/photon.js
@@ -44,8 +44,13 @@
 	/**
 	 * Check both when page loads, and when IS is triggered.
 	 */
-	if ( typeof window !== 'undefined' ) {
-		window.wp.domReady( restore_dims );
+	if ( typeof window !== 'undefined' && typeof document !== 'undefined' ) {
+		// `DOMContentLoaded` may fire before the script has a chance to run
+		if ( document.readyState === 'loading' ) {
+			document.addEventListener( 'DOMContentLoaded', restore_dims );
+		} else {
+			restore_dims();
+		}
 	}
 
 	document.body.addEventListener( 'post-load', restore_dims );