Merge pull request #2601 from yougotwill/zoom_controls

Zoom Controls
BoostIO · Nov 25, 2018 · c7bae93 · c7bae93
2 parents 938cf23 + d3a6ff6
commit c7bae93
Show file tree

Hide file tree

Showing 2 changed files with 52 additions and 3 deletions.
diff --git 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
@@ -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')
+      }
+    },
+    {
+      label: 'Zoom In',
+      accelerator: macOS ? 'CommandOrControl+=' : 'Control+=',
+      click () {
+        mainWindow.webContents.send('status:zoomin')
+      }
     },
     {
-      role: 'zoomout'
+      label: 'Zoom Out',
+      accelerator: macOS ? 'CommandOrControl+-' : 'Control+-',
+      click () {
+        mainWindow.webContents.send('status:zoomout')
+      }
     }
   ]
 }