chore: Implement double-click notification management in background and content scripts

parlandin · parlandin · commit bc1b315da31d · 2025-05-31T16:37:41.000-03:00
diff --git a/scripts/background.js b/scripts/background.js
@@ -51,6 +51,8 @@ const MESSAGE_ACTIONS = {
   DELETE_OBJECT_URL: "deleteObjectURL",
 };
 
+let doubleClickNotificationShown = false;
+
 // ============================================================================
 // CONTEXT MENU MANAGEMENT
 // ============================================================================
@@ -323,6 +325,17 @@ async function handleRuntimeMessage(message, sender, sendResponse) {
       });
       break;
 
+    case "checkDoubleClickNotification":
+      return Promise.resolve({ alreadyShown: doubleClickNotificationShown });
+
+    case "markDoubleClickNotificationShown":
+      doubleClickNotificationShown = true;
+      return Promise.resolve({ success: true });
+
+    case "clearDoubleClickNotification":
+      doubleClickNotificationShown = false;
+      return Promise.resolve({ success: true });
+
     default:
       return false;
   }
diff --git a/scripts/content_script.js b/scripts/content_script.js
@@ -118,21 +118,23 @@ async function handleImageDoubleClickValidation() {
   if (isEnabled) {
     return isEnabled;
   } else {
-    let notificationCount = parseInt(
-      sessionStorage.getItem("doubleClickNotificationCount") || "0"
-    );
+    // Query the background script for notification state
+    const response = await browser.runtime.sendMessage({
+      action: "checkDoubleClickNotification",
+    });
 
-    if (notificationCount < 2) {
+    if (!response.alreadyShown) {
       const errorMessage =
         "Download por duplo clique desativado. Acesse as configurações.";
       showToast(errorMessage, TOAST_TYPES.ERROR);
 
-      notificationCount++;
-      sessionStorage.setItem(
-        "doubleClickNotificationCount",
-        notificationCount.toString()
-      );
+      // Tell background script we've shown the notification
+      browser.runtime.sendMessage({
+        action: "markDoubleClickNotificationShown",
+      });
     }
+
+    return false;
   }
 }
 
diff --git a/scripts/index.js b/scripts/index.js
@@ -26,6 +26,10 @@ function handleSettingsButtonClick() {
 async function updateDoubleClickSetting(isEnabled) {
   try {
     await browser.storage.local.set({ doubleClickEnabled: isEnabled });
+
+    await browser.runtime.sendMessage({
+      action: "clearDoubleClickNotification",
+    });
   } catch (error) {
     console.error("Error updating double click setting:", error);
   }
diff --git a/scripts/options.js b/scripts/options.js
@@ -83,6 +83,10 @@ async function addNewSuffix(suffixes) {
 async function updateDoubleClickSetting(isEnabled) {
   try {
     await browser.storage.local.set({ doubleClickEnabled: isEnabled });
+
+    await browser.runtime.sendMessage({
+      action: "clearDoubleClickNotification",
+    });
   } catch (error) {
     console.error("Error updating double click setting:", error);
   }
