feat: add support to decide if fullscreen ignores all commands

zzogem · Sep 18, 2024 · 6d74141 · 6d74141
1 parent ae5d3b6
commit 6d74141
Show file tree

Hide file tree

Showing 8 changed files with 24 additions and 7 deletions.
diff --git a/chrome/options.html b/chrome/options.html
@@ -102,7 +102,8 @@ <h4>Settings</h4>
         	<label><input type="checkbox" id="scrollpause"><span class="label-text">Enable auto pause when out of viewport</span></label>
 			<label><input type="checkbox" id="cursorTracking"><span class="label-text">Enable cursor on window tracking</span></label>
 			<label><input type="checkbox" id="manualPause"><span class="label-text">Manual pause disables auto resume</span></label>
-			<label><input type="checkbox" id="disabled"><span class="label-text">Disable auto pause/resume</span></label>
+			<label><input type="checkbox" id="disableOnFullscreen"><span class="label-text">Disable on full screen</span></label>
+			<label><input type="checkbox" id="disabled"><span class="label-text">Disable extension</span></label>
 		</div>
 		<hr>
 		<div class="container">

diff --git a/chrome/options.js b/chrome/options.js
@@ -10,6 +10,7 @@ const options = {
   cursorTracking: false,
   manualPause: true,
   debugMode: false,
+  disableOnFullscreen: false,
 };
 
 const hosts = chrome.runtime.getManifest().host_permissions;

diff --git a/chrome/yt.js b/chrome/yt.js
@@ -18,6 +18,7 @@ let options = {
   disabled: false,
   cursorTracking: false,
   debugMode: false,
+  disableOnFullscreen: false,
 };
 
 const hosts = chrome.runtime.getManifest().host_permissions;
@@ -41,6 +42,7 @@ function refresh_settings() {
       options.focusresume = false;
       options.cursorTracking = false;
       options.debugMode = false;
+      options.disableOnFullscreen = true;
       for (var host of hosts) {
         options[host] = false;
       }

diff --git a/chrome/yt_auto_pause.js b/chrome/yt_auto_pause.js
@@ -15,6 +15,7 @@ if (window.ytAutoPauseInjected !== true) {
     cursorTracking: false,
     manualPause: true,
     debugMode: false,
+    disableOnFullscreen: false,
   };
 
   function debugLog(message) {
@@ -39,6 +40,7 @@ if (window.ytAutoPauseInjected !== true) {
         options.focusresume = false;
         options.cursorTracking = false;
         options.debugMode = false;
+        options.disableOnFullscreen = true;
         for (var host of hosts) {
           options[host] = false;
         }
@@ -145,6 +147,11 @@ if (window.ytAutoPauseInjected !== true) {
     }
     debugLog(`Received message: ${JSON.stringify(request)}`);
 
+    if (document.fullscreenElement && options.disableOnFullscreen) {
+      debugLog(`Document is in fullscreen mode, ignoring all commands`);
+      return true;
+    }
+
     const videoElements = document.getElementsByTagName("video");
     const iframeElements = document.getElementsByTagName("iframe");
 
@@ -169,11 +176,6 @@ if (window.ytAutoPauseInjected !== true) {
 
     for (let i = 0; i < videoElements.length; i++) {
       try {
-        if (document.fullscreenElement) {
-          debugLog(`Document is in fullscreen mode, ignoring all commands`);
-          break;
-        }
-
         if (request.action === "stop" && !manuallyPaused) {
           automaticallyPaused = true;
           videoElements[i].pause();

diff --git a/firefox/options.html b/firefox/options.html
@@ -102,7 +102,8 @@ <h4>Settings</h4>
         	<label><input type="checkbox" id="scrollpause"><span class="label-text">Enable auto pause when out of viewport</span></label>
 			<label><input type="checkbox" id="cursorTracking"><span class="label-text">Enable cursor on window tracking</span></label>
 			<label><input type="checkbox" id="manualPause"><span class="label-text">Manual pause disables auto resume</span></label>
-			<label><input type="checkbox" id="disabled"><span class="label-text">Disable auto pause/resume</span></label>
+			<label><input type="checkbox" id="disableOnFullscreen"><span class="label-text">Disable on full screen</span></label>
+			<label><input type="checkbox" id="disabled"><span class="label-text">Disable extension</span></label>
 		</div>
 		<hr>
 		<div class="container">

diff --git a/firefox/options.js b/firefox/options.js
@@ -10,6 +10,7 @@ const options = {
   cursorTracking: false,
   manualPause: true,
   debugMode: false,
+  disableOnFullscreen: false,
 };
 
 const hosts = browser.runtime.getManifest().host_permissions;

diff --git a/firefox/yt.js b/firefox/yt.js
@@ -18,6 +18,7 @@ let options = {
   disabled: false,
   cursorTracking: false,
   debugMode: false,
+  disableOnFullscreen: false,
 };
 
 const hosts = browser.runtime.getManifest().host_permissions;
@@ -43,6 +44,7 @@ function refresh_settings() {
       options.focusresume = false;
       options.cursorTracking = false;
       options.debugMode = false;
+      options.disableOnFullscreen = true;
       for (var host of hosts) {
         options[host] = false;
       }

diff --git a/firefox/yt_auto_pause.js b/firefox/yt_auto_pause.js
@@ -15,6 +15,7 @@ if (window.ytAutoPauseInjected !== true) {
     cursorTracking: false,
     manualPause: true,
     debugMode: false,
+    disableOnFullscreen: false,
   };
 
   function debugLog(message) {
@@ -39,6 +40,7 @@ if (window.ytAutoPauseInjected !== true) {
         options.focusresume = false;
         options.cursorTracking = false;
         options.debugMode = false;
+        options.disableOnFullscreen = true;
         for (var host of hosts) {
           options[host] = false;
         }
@@ -143,7 +145,12 @@ if (window.ytAutoPauseInjected !== true) {
     if (!("action" in request)) {
       return false;
     }
+
     debugLog(`Received message: ${JSON.stringify(request)}`);
+    if (document.fullscreenElement && options.disableOnFullscreen) {
+      debugLog(`Document is in fullscreen mode, ignoring all commands`);
+      return true;
+    }
 
     const videoElements = document.getElementsByTagName("video");
     const iframeElements = document.getElementsByTagName("iframe");