Merge pull request #4093 from luixxiul/bringAllToFront-macOS

Show "Bring All to Front" only on macOS
brave · Sep 19, 2016 · c5ac72c · c5ac72c
2 parents 3fbf38d + 56e920c
commit c5ac72c
Showing 1 changed file with 14 additions and 7 deletions.
diff --git a/app/browser/menu.js b/app/browser/menu.js
@@ -405,7 +405,7 @@ const createBookmarksSubmenu = () => {
 }
 
 const createWindowSubmenu = () => {
-  return [
+  const submenu = [
     {
       label: locale.translation('minimize'),
       accelerator: 'CmdOrCtrl+M',
@@ -438,13 +438,20 @@ const createWindowSubmenu = () => {
     CommonMenu.separatorMenuItem,
     CommonMenu.bookmarksManagerMenuItem(),
     CommonMenu.downloadsMenuItem(),
-    CommonMenu.passwordsMenuItem(),
-    CommonMenu.separatorMenuItem,
-    {
-      label: locale.translation('bringAllToFront'),
-      role: 'front'
-    }
+    CommonMenu.passwordsMenuItem()
   ]
+
+  if (isDarwin) {
+    submenu.push(
+      CommonMenu.separatorMenuItem,
+      {
+        label: locale.translation('bringAllToFront'),
+        role: 'front'
+      }
+    )
+  }
+
+  return submenu
 }
 
 const createHelpSubmenu = () => {