Merge pull request #1754 from yougotwill/crossplatform_fullscreen_shortcuts

Fullscreen shortcut for non macOS

browser/main/NoteList/index.js

@@ -282,8 +282,8 @@ class NoteList extends React.Component {
       ee.emit('detail:focus')
     }
 
-    // F or S key
+    // L or S key
-    if (e.keyCode === 70 || e.keyCode === 83) {
+    if (e.keyCode === 76 || e.keyCode === 83) {
       e.preventDefault()
       ee.emit('top:focus-search')
     }

lib/main-menu.js

@@ -252,10 +252,20 @@ const view = {
     },
     {
       label: 'Focus Search',
-      accelerator: 'Control+S',
+      accelerator: 'CommandOrControl+Shift+L',
       click () {
         mainWindow.webContents.send('top:focus-search')
       }
+    },
+    {
+      type: 'separator'
+    },
+    {
+      label: 'Toggle Full Screen',
+      accelerator: macOS ? 'Command+Control+F' : 'F11',
+      click () {
+        mainWindow.setFullScreen(!mainWindow.isFullScreen())
+      }
     }
   ]
 }
@@ -265,7 +275,7 @@ let editorFocused
 // Define extra shortcut keys
 mainWindow.webContents.on('before-input-event', (event, input) => {
   // Synonyms for Search (Find)
-  if (input.control && input.key === 'f' && input.type === 'keyDown') {
+  if (input.control && input.key === 'l' && input.type === 'keyDown') {
     if (!editorFocused) {
       mainWindow.webContents.send('top:focus-search')
       event.preventDefault()
@@ -285,11 +295,6 @@ const window = {
       accelerator: 'Command+M',
       selector: 'performMiniaturize:'
     },
-    {
-      label: 'Toggle Full Screen',
-      accelerator: 'Command+Control+F',
-      selector: 'toggleFullScreen:'
-    },
     {
       label: 'Close',
       accelerator: 'Command+W',