Merge pull request #5671 from Ihyatt/feature/removed-redundant-menu-item

Remove Open in new tab from context menu while in a private browsing tab
brave · Nov 16, 2016 · 78a2fb6 · 78a2fb6
2 parents e1b985d + adbe7ae
commit 78a2fb6
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/js/contextMenus.js b/js/contextMenus.js
@@ -902,7 +902,10 @@ function mainTemplateInit (nodeProps, frame) {
   const isAboutPage = aboutUrls.has(frame.get('location'))
 
   if (isLink) {
-    template.push(openInNewTabMenuItem(nodeProps.linkURL, frame.get('isPrivate'), frame.get('partitionNumber'), frame.get('key')),
+    if (!frame.get('isPrivate')) {
+      template.push(openInNewTabMenuItem(nodeProps.linkURL, frame.get('isPrivate'), frame.get('partitionNumber'), frame.get('key')))
+    }
+    template.push(
       openInNewPrivateTabMenuItem(nodeProps.linkURL, frame.get('key')),
       openInNewWindowMenuItem(nodeProps.linkURL, frame.get('isPrivate'), frame.get('partitionNumber')),
       CommonMenu.separatorMenuItem,