fixed multiple notification windows when executing batch

1 files changed, 10 insertions, 1 deletions

diff --git a/app/scripts/background.js b/app/scripts/background.js
index 601ae0372..6cb80244f 100644
--- a/app/scripts/background.js
+++ b/app/scripts/background.js
@@ -40,6 +40,7 @@ const isIE = !!document.documentMode
 const isEdge = !isIE && !!window.StyleMedia
 
 let popupIsOpen = false
+let notifcationIsOpen = false;
 let openMetamaskTabsIDs = {}
 
 // state persistence
@@ -136,6 +137,11 @@ function setupController (initState) {
           }
         })
       }
+      if (remotePort.name === 'notification') {
+        endOfStream(portStream, () => {
+          notifcationIsOpen = false
+        });
+      }
     } else {
       // communication with page
       const originDomain = urlUtil.parse(remotePort.sender.url).hostname
@@ -178,7 +184,10 @@ function setupController (initState) {
 function triggerUi () {
   extension.tabs.query({ active: true }, (tabs) => {
     const currentlyActiveMetamaskTab = tabs.find(tab => openMetamaskTabsIDs[tab.id])
-    if (!popupIsOpen && !currentlyActiveMetamaskTab) notificationManager.showPopup()
+    if (!popupIsOpen && !currentlyActiveMetamaskTab) notificationManager.showPopup((notification) => {
+      notifcationIsOpen = notification;
+    });
+    notifcationIsOpen = true;
   })
 }