diff --git a/CHANGELOG.md b/CHANGELOG.md index 72b5ea72..37598b19 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -4,8 +4,11 @@ ### Improvements +#### Windows +- Use the application icon for desktop notifications in Windows 10. + #### Linux - - Added an option to make the taskbar icon flash on new messages +- Added an option to make the taskbar icon flash on new messages ### Bug Fixes diff --git a/src/browser/js/notification.js b/src/browser/js/notification.js index 9eb14cc6..c3a49d6f 100644 --- a/src/browser/js/notification.js +++ b/src/browser/js/notification.js @@ -1,9 +1,15 @@ const OriginalNotification = Notification; +const appIconURL = `file:///${require('electron').remote.app.getAppPath()}/resources/appicon.png`; + function override(eventHandlers) { Notification = function(title, options) { - // Notification Center shows app's icon, so there were two icons on the notification. - if (process.platform === 'darwin') { + if (process.platform === 'win32') { + // Replace with application icon. + options.icon = appIconURL; + } + else if (process.platform === 'darwin') { + // Notification Center shows app's icon, so there were two icons on the notification. delete options.icon; } this.notification = new OriginalNotification(title, options);