Show menu on only macOS

Author: hidakatsuya

app/main.js

@@ -24,57 +24,67 @@ function createWindow () {
     shell.openExternal(url)
   })
 
-  const template = [
-    {
+  win.on('closed', () => win = null)
+
+  setupMenu()
+}
+
+app.on('ready', createWindow)
+app.on('window-all-closed', () => {
+  if (process.platform !== 'darwin') {
+    app.quit()
+  }
+})
+app.on('activate', () => {
+  if (win === null) {
+    createWindow()
+  }
+})
+
+function setupMenu () {
+  const menus = []
+
+  if (process.platform === 'darwin') {
+    menus.push({
       label: 'Application',
       submenu: [
-        { label: 'Quit', accelerator: 'Command+Q', click: () => { app.quit() } }
+        { role: 'quit' }
       ]
-    },
-    {
+    })
+
+    menus.push({
       label: 'Edit',
       submenu: [
-        { label: 'Undo', accelerator: 'CmdOrCtrl+Z', selector: 'undo:' },
-        { label: 'Redo', accelerator: 'Shift+CmdOrCtrl+Z', selector: 'redo:' },
+        { role: 'undo' },
+        { role: 'redo' },
         { type: 'separator' },
-        { label: 'Cut', accelerator: 'CmdOrCtrl+X', selector: 'cut:' },
-        { label: 'Copy', accelerator: 'CmdOrCtrl+C', selector: 'copy:' },
-        { label: 'Paste', accelerator: 'CmdOrCtrl+V', selector: 'paste:' },
-        { label: 'Select All', accelerator: 'CmdOrCtrl+A', selector: 'selectAll:' }
+        { role: 'cut' },
+        { role: 'copy' },
+        { role: 'paste' },
+        { role: 'selectall' }
       ]
-    }
-  ]
+    })
+  }
 
   if (process.env.NODE_ENV === 'development') {
-    template.push({
+    menus.push({
       label: 'View',
       submenu: [
         {
           label: 'Reload',
           accelerator: 'Command+R',
-          click: (_, activeWin) => activeWin.reload()
+          click: () => win.reload()
         },
         {
           label: 'Toggle Developer Tools',
           accelerator: 'Alt+Command+I',
-          click: (_, activeWin) => activeWin.toggleDevTools()
+          click: () => win.toggleDevTools()
         },
       ]
     })
   }
-  Menu.setApplicationMenu(Menu.buildFromTemplate(template))
 
-  win.on('closed', () => win = null)
-}
-
-app.on('ready', createWindow)
-app.on('window-all-closed', () => {
-  if (process.platform !== 'darwin') {
-    app.quit()
-  }
-})
-app.on('activate', () => {
-  if (win === null) {
-    createWindow()
+  if (menus.length > 0) {
+    Menu.setApplicationMenu(Menu.buildFromTemplate(menus))
   }
-})
+}