From e2c7a8c3840abb0f1c1e6128f19205e559a61ae8 Mon Sep 17 00:00:00 2001
From: William Grant <willmgrant@gmail.com>
Date: Sun, 11 Nov 2018 12:59:50 +0200
Subject: [PATCH] updated the zoom logic for the zoom in/out menu items so that
 if you zoom it reflects in the status bar and the config. Also added an
 actual size / zoom reset menu item for convenience.

---
 browser/main/StatusBar/index.js | 35 +++++++++++++++++++++++++++++++++
 lib/main-menu.js                | 20 ++++++++++++++++---
 2 files changed, 52 insertions(+), 3 deletions(-)

diff --git a/browser/main/StatusBar/index.js b/browser/main/StatusBar/index.js
index 8b48e3d3..c99bf036 100644
--- a/browser/main/StatusBar/index.js
+++ b/browser/main/StatusBar/index.js
@@ -5,6 +5,7 @@ import styles from './StatusBar.styl'
 import ZoomManager from 'browser/main/lib/ZoomManager'
 import i18n from 'browser/lib/i18n'
 import context from 'browser/lib/context'
+import EventEmitter from 'browser/main/lib/eventEmitter'
 
 const electron = require('electron')
 const { remote, ipcRenderer } = electron
@@ -13,6 +14,26 @@ const { dialog } = remote
 const zoomOptions = [0.8, 0.9, 1, 1.1, 1.2, 1.3, 1.4, 1.5, 1.6, 1.7, 1.8, 1.9, 2.0]
 
 class StatusBar extends React.Component {
+
+  constructor (props) {
+    super(props)
+    this.handleZoomInMenuItem = this.handleZoomInMenuItem.bind(this)
+    this.handleZoomOutMenuItem = this.handleZoomOutMenuItem.bind(this)
+    this.handleZoomResetMenuItem = this.handleZoomResetMenuItem.bind(this)
+  }
+
+  componentDidMount () {
+    EventEmitter.on('status:zoomin', this.handleZoomInMenuItem)
+    EventEmitter.on('status:zoomout', this.handleZoomOutMenuItem)
+    EventEmitter.on('status:zoomreset', this.handleZoomResetMenuItem)
+  }
+
+  componentWillUnmount () {
+    EventEmitter.off('status:zoomin', this.handleZoomInMenuItem)
+    EventEmitter.off('status:zoomout', this.handleZoomOutMenuItem)
+    EventEmitter.off('status:zoomreset', this.handleZoomResetMenuItem)
+  }
+
   updateApp () {
     const index = dialog.showMessageBox(remote.getCurrentWindow(), {
       type: 'warning',
@@ -48,6 +69,20 @@ class StatusBar extends React.Component {
     })
   }
 
+  handleZoomInMenuItem () {
+    const zoomFactor = ZoomManager.getZoom() + 0.1
+    this.handleZoomMenuItemClick(zoomFactor)
+  }
+
+  handleZoomOutMenuItem () {
+    const zoomFactor = ZoomManager.getZoom() - 0.1
+    this.handleZoomMenuItemClick(zoomFactor)
+  }
+
+  handleZoomResetMenuItem () {
+    this.handleZoomMenuItemClick(1.0)
+  }
+
   render () {
     const { config, status } = this.context
 
diff --git a/lib/main-menu.js b/lib/main-menu.js
index fed5eb15..6f366b63 100644
--- a/lib/main-menu.js
+++ b/lib/main-menu.js
@@ -310,11 +310,25 @@ const view = {
       type: 'separator'
     },
     {
-      role: 'zoomin',
-      accelerator: macOS ? 'CommandOrControl+Plus' : 'Control+='
+      label: 'Actual Size',
+      accelerator: macOS ? 'CommandOrControl+0' : 'Control+0',
+      click () {
+        mainWindow.webContents.send('status:zoomreset')
+      }
     },
     {
-      role: 'zoomout'
+      label: 'Zoom In',
+      accelerator: macOS ? 'CommandOrControl+=' : 'Control+=',
+      click () {
+        mainWindow.webContents.send('status:zoomin')
+      }
+    },
+    {
+      label: 'Zoom Out',
+      accelerator: macOS ? 'CmdOrCtrl+-' : 'Control+-',
+      click () {
+        mainWindow.webContents.send('status:zoomout')
+      }
     }
   ]
 }