Prevent exception when no active window

Author: Yukaii

ipc/consumer.js

@@ -7,28 +7,36 @@ const url = require('url')
 const fetch = require('node-fetch')
 const semver = require('semver')
 
+function sendToWebContent (event) {
+  const win = BrowserWindow.getFocusedWindow()
+  const webContent = win && win.webContents
+  if (webContent) {
+    webContent.send(event)
+  }
+}
+
 module.exports = function (commandId, args = {}) {
   switch (commandId) {
     case 'createWindow':
       createWindow(args)
       break
     case 'refreshWindow':
-      BrowserWindow.getFocusedWindow().webContents.send('web:refresh')
+      sendToWebContent('web:refresh')
       break
     case 'learnMore':
       shell.openExternal('https://hackmd.io')
       break
     case 'goForward':
-      BrowserWindow.getFocusedWindow().webContents.send('web:go-forward')
+      sendToWebContent('web:go-forward')
       break
     case 'goBack':
-      BrowserWindow.getFocusedWindow().webContents.send('web:go-back')
+      sendToWebContent('web:go-back')
       break
     case 'configServerUrl':
-      BrowserWindow.getFocusedWindow().webContents.send('config-serverurl')
+      sendToWebContent('config-serverurl')
       break
     case 'openFromUrl':
-      BrowserWindow.getFocusedWindow().webContents.send('open-from-url')
+      sendToWebContent('open-from-url')
       break
     case 'checkVersion':
       return fetch(url.resolve(getServerUrl(), '/status')).then(response => {
@@ -44,10 +52,10 @@ module.exports = function (commandId, args = {}) {
         }
       }).catch(err => console.log(err))
     case 'copyUrl':
-      BrowserWindow.getFocusedWindow().webContents.send('copy-url')
+      sendToWebContent('copy-url')
       break
     case 'toggleSearch':
-      BrowserWindow.getFocusedWindow().webContents.send('toggle-search')
+      sendToWebContent('toggle-search')
       break
     default:
       break

main.js

@@ -1,4 +1,4 @@
-const { app, Menu } = require('electron')
+const { app, Menu, BrowserWindow } = require('electron')
 const path = require('path')
 const menu = require('./menu')
 const { createWindow } = require('./window')
@@ -22,5 +22,7 @@ app.on('window-all-closed', () => {
 })
 
 app.on('activate', () => {
-  initializeApp()
+  if (process.platform === 'darwin' && BrowserWindow.getAllWindows().length === 0) {
+    initializeApp()
+  }
 })