Show menu bar by default on Linux and Windows and add option to disab…

…le that (#302)
sindresorhus · Dec 17, 2017 · 293aa7e · 293aa7e
1 parent 100a437
commit 293aa7e
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/config.js b/config.js
@@ -21,6 +21,7 @@ module.exports = new Store({
 		},
 		confirmImagePaste: true,
 		useWorkChat: false,
-		sidebarHidden: false
+		sidebarHidden: false,
+		autoHideMenuBar: false
 	}
 });
diff --git a/index.js b/index.js
@@ -156,7 +156,7 @@ function createMainWindow() {
 		alwaysOnTop: config.get('alwaysOnTop'),
 		// Temp workaround for macOS High Sierra, see #295
 		titleBarStyle: process.platform === 'darwin' && Number(require('os').release().split('.')[0]) >= 17 ? null : 'hidden-inset',
-		autoHideMenuBar: true,
+		autoHideMenuBar: config.get('autoHideMenuBar'),
 		darkTheme: isDarkMode, // GTK+3
 		webPreferences: {
 			preload: path.join(__dirname, 'browser.js'),

diff --git a/menu.js b/menu.js
@@ -459,6 +459,16 @@ const otherTpl = [
 					config.set('block.typingIndicator', item.checked);
 				}
 			},
+			{
+				type: 'checkbox',
+				label: 'Auto Hide Menu Bar',
+				checked: config.get('autoHideMenuBar'),
+				click(item, focusedWindow) {
+					config.set('autoHideMenuBar', item.checked);
+					focusedWindow.setAutoHideMenuBar(item.checked);
+					focusedWindow.setMenuBarVisibility(!item.checked);
+				}
+			},
 			{
 				type: 'separator'
 			},