[FIX] Handles icon visibility and fixes context menu with undefined ID

team-underscore · Aug 12, 2019 · 137dc4b · 137dc4b
1 parent 2ed4a12
commit 137dc4b
Show file tree

Hide file tree

Showing 6 changed files with 5 additions and 5 deletions.
diff --git a/background.js b/background.js
@@ -1,5 +1,5 @@
 let ContextMenuItem = {
-  "id": `${Date().now}`,
+  "id": `RevealURL_Extension`,
   "title": "RevealURL",
   "contexts": ["link"]
 }
@@ -9,7 +9,7 @@ chrome.contextMenus.create(ContextMenuItem);
 // Creates a Status notification
 chrome.contextMenus.onClicked.addListener(function (info) {
 
-  createNotification('RevealURL', 'Please wait while we get the data', 0)
+  createNotification('RevealURL', 'Please wait while we get the data', 1)
 
   // API call
   var xhr = new XMLHttpRequest();
@@ -22,7 +22,7 @@ chrome.contextMenus.onClicked.addListener(function (info) {
       if (xhr.status === 200) {
         handleCallback(xhr.response)
       } else {
-        createNotification('ERROR', 'Please try again later  😓', 1)
+        createNotification('ERROR', 'Please try again later  😓', 2)
       }
     }
   }
@@ -65,7 +65,7 @@ function handleCallback(resp) {
     url = '😑 Thats not a short URL'
   }
 
-  createNotification(title, url, 1)
+  createNotification(title, url, 2)
 }
 
 function createNotification(title, message, priority) {

diff --git a/images/icon_128.png b/images/icon_128.png
diff --git a/images/icon_16.png b/images/icon_16.png
diff --git a/images/icon_32.png b/images/icon_32.png
diff --git a/images/icon_48.png b/images/icon_48.png
diff --git a/manifest.json b/manifest.json
@@ -1,6 +1,6 @@
 {
   "name": "RevealURL",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "description": "Validate any shortURL with RevealURL ",
   "permissions": [
     "contextMenus",