fixed problems with the enabled list

apertium · Jun 21, 2021 · 38fd57b · 38fd57b
1 parent e0f3c75
commit 38fd57b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/src/background/background.js b/src/background/background.js
@@ -13,13 +13,13 @@ browser.runtime.onMessage.addListener(function(request, sender, sendResponse) {
 browser.runtime.onInstalled.addListener(function () {
     browser.contextMenus.create({
         "id": "enableTranslation",
-        "title": "Toggle Hover-On Translation",
+        "title": "Enable Hover-On Translation",
         "documentUrlPatterns": ["<all_urls>"],
         "contexts": ["page"],
     });
 });
 
-chrome.contextMenus.onClicked.addListener(function (info, tab) {
+browser.contextMenus.onClicked.addListener(function (info, tab) {
     if (info.menuItemId === "enableTranslation") {
         addToEnabledWebsiteList(getGlobalSettings(), tab.url);
     }

diff --git a/src/background/storage.js b/src/background/storage.js
@@ -242,16 +242,18 @@ function saveEnabledWebsiteList(settings, newList) {
     saveGlobalSettings(settings);
 }
 
-function removeFromEnabledWebsiteList(settings, url){
+function removeFromEnabledWebsiteList(settings, hostname){
     let list = getEnabledWebsiteList();
 
-    if(list.includes(url)) {
-        let index = list.indexOf(url);
+    if(list.includes(hostname)) {
+        let index = list.indexOf(hostname);
         if (index !== -1) {
             list.splice(index, 1);
         }
     }
 
+    console.log(list);
+
     saveEnabledWebsiteList(settings, list);
 }