From f57e2a94efb807eebe128e14b48e13e8d696117b Mon Sep 17 00:00:00 2001 From: Yuya Ochiai Date: Wed, 23 Dec 2015 17:42:21 +0900 Subject: [PATCH] Just focus mainWindow when notification is clicked on Windows BrowserWindow.show() breaks Aero Snap state. --- src/browser/webview/mattermost.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/browser/webview/mattermost.js b/src/browser/webview/mattermost.js index f56c6526..d2c95725 100644 --- a/src/browser/webview/mattermost.js +++ b/src/browser/webview/mattermost.js @@ -52,7 +52,13 @@ function overrideNotification() { }; Notification.prototype.__defineSetter__('onclick', function(callback) { this.notification.onclick = function() { - electron.remote.getCurrentWindow().show(); + if (process.platform === 'win32') { + // show() breaks Aero Snap state. + electron.remote.getCurrentWindow().focus(); + } + else { + electron.remote.getCurrentWindow().show(); + } ipc.sendToHost('onNotificationClick'); callback(); };