revert to original change

Author: Judahmeek

packages/react-on-rails/src/pageLifecycle.ts

@@ -59,7 +59,7 @@ function setupPageNavigationListeners(): void {
 }
 
 let isPageLifecycleInitialized = false;
-function initializePageEventListeners(): void {
+function onPageReady(callback: () => void) {
   if (typeof window === 'undefined') return;
 
   if (isPageLifecycleInitialized) {
@@ -68,9 +68,12 @@ function initializePageEventListeners(): void {
   isPageLifecycleInitialized = true;
 
   if (document.readyState === 'complete') {
-    setupPageNavigationListeners();
+    callback();
   } else {
-    document.addEventListener('load', setupPageNavigationListeners);
+    document.addEventListener('readystatechange', function onReadyStateChange() {
+      onPageReady(callback);
+      document.removeEventListener('readystatechange', onReadyStateChange);
+    });
   }
 }
 
@@ -79,13 +82,13 @@ export function onPageLoaded(callback: PageLifecycleCallback): void {
     void callback();
   }
   pageLoadedCallbacks.add(callback);
-  initializePageEventListeners();
+  onPageReady(setupPageNavigationListeners);
 }
 
 export function onPageUnloaded(callback: PageLifecycleCallback): void {
   if (currentPageState === 'unload') {
     void callback();
   }
   pageUnloadedCallbacks.add(callback);
-  initializePageEventListeners();
+  onPageReady(setupPageNavigationListeners);
 }

packages/react-on-rails/tests/pageLifecycle.test.js

@@ -91,7 +91,7 @@ describe('pageLifecycle', () => {
     // Should not call callback immediately since readyState is 'loading'
     expect(callback).not.toHaveBeenCalled();
     // Verify that a DOMContentLoaded listener was added when readyState is 'loading'
-    expect(addEventListenerSpy).toHaveBeenCalledWith('load', expect.any(Function));
+    expect(addEventListenerSpy).toHaveBeenCalledWith('readystatechange', expect.any(Function));
   });
 
   it('should wait for readystatechange when document.readyState is "loading"', () => {
@@ -104,7 +104,7 @@ describe('pageLifecycle', () => {
     // Should not call callback immediately since readyState is 'loading'
     expect(callback).not.toHaveBeenCalled();
     // Verify that a DOMContentLoaded listener was added when readyState is 'loading'
-    expect(addEventListenerSpy).toHaveBeenCalledWith('load', expect.any(Function));
+    expect(addEventListenerSpy).toHaveBeenCalledWith('readystatechange', expect.any(Function));
   });
 
   describe('with Turbo navigation library', () => {