From 82408e8e2930751019129e341fb74187b12e0e67 Mon Sep 17 00:00:00 2001 From: Jason Blais Date: Mon, 19 Sep 2016 09:21:56 -0400 Subject: [PATCH] Updates to menu bar Proposed updates to menu bar: - Change 'Help > Mattermost Docs' to 'Help > Learn More' and link to newly submitted desktop docs on docs.mattermost.com [to be merged within the next 48 hours] - Change 'Settings' and 'Learn More' to include '...' as they open a new page. This is standard to other desktop apps --- src/main/menus/app.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/main/menus/app.js b/src/main/menus/app.js index 8138d271..4e853777 100644 --- a/src/main/menus/app.js +++ b/src/main/menus/app.js @@ -36,7 +36,7 @@ var createTemplate = function(mainWindow, config) { }, separatorItem, { role: 'quit' }] : [{ - label: 'Settings', + label: 'Settings...', accelerator: 'CmdOrCtrl+,', click: function(item, focusedWindow) { mainWindow.loadURL('file://' + __dirname + '/browser/settings.html'); @@ -229,9 +229,9 @@ var createTemplate = function(mainWindow, config) { template.push({ label: '&Help', submenu: [{ - label: `${app_name} Docs`, + label: `Learn More...`, click: function() { - electron.shell.openExternal('http://docs.mattermost.com'); + electron.shell.openExternal('https://docs.mattermost.com/help/apps/desktop-guide.html'); } }, { type: 'separator'