move isPageScrolled method to Utils class

wp-media · Oct 15, 2024 · 4f760de · 4f760de
1 parent 8544b7d
commit 4f760de
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/BeaconManager.js b/src/BeaconManager.js
@@ -22,7 +22,7 @@ class BeaconManager {
             return;
         }
 
-        if (this._isPageScrolled()) {
+        if (BeaconUtils.isPageScrolled()) {
             this.logger.logMessage('Bailing out because the page has been scrolled');
             this._finalize();
             return;
@@ -149,10 +149,6 @@ class BeaconManager {
         this._saveFinalResultIntoDB();
     }
 
-    _isPageScrolled() {
-        return window.pageYOffset > 0 || document.documentElement.scrollTop > 0;
-    }
-
     _finalize() {
         const beaconscript = document.querySelector('[data-name="wpr-wpr-beacon"]');
         beaconscript.setAttribute('beacon-completed', 'true');

diff --git a/src/Utils.js b/src/Utils.js
@@ -35,6 +35,10 @@ class BeaconUtils {
         );
     }
 
+    static isPageScrolled() {
+        return window.pageYOffset > 0 || document.documentElement.scrollTop > 0;
+    }
+
 }
 
 export default BeaconUtils;