From 93a725d453be676c88cb66ea5adb5616751aadc3 Mon Sep 17 00:00:00 2001 From: Jim Graham Date: Wed, 25 Feb 2015 17:11:17 -0500 Subject: [PATCH] Remove dialog icon from updater atom/atom#5670 Remove the update icon on windows when showing the update modal dialog. Keep the icon for linux. This appears to be the only file outside of `build/` that uses the `'atom.png'` file. --- src/browser/auto-update-manager.coffee | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/browser/auto-update-manager.coffee b/src/browser/auto-update-manager.coffee index dfbdacfc4..25cf0dc18 100644 --- a/src/browser/auto-update-manager.coffee +++ b/src/browser/auto-update-manager.coffee @@ -17,10 +17,12 @@ class AutoUpdateManager constructor: (@version) -> @state = IdleState + @iconPath = path.resolve(__dirname, '..', '..', 'resources', 'atom.png'); if process.platform is 'win32' # Squirrel for Windows can't handle query params # https://github.com/Squirrel/Squirrel.Windows/issues/132 @feedUrl = 'https://atom.io/api/updates' + @iconPath = null; else @feedUrl = "https://atom.io/api/updates?version=#{@version}" @@ -89,7 +91,7 @@ class AutoUpdateManager dialog.showMessageBox type: 'info' buttons: ['OK'] - icon: path.resolve(__dirname, '..', '..', 'resources', 'atom.png') + icon: @iconPath message: 'No update available.' title: 'No Update Available' detail: "Version #{@version} is the latest version." @@ -100,7 +102,7 @@ class AutoUpdateManager dialog.showMessageBox type: 'warning' buttons: ['OK'] - icon: path.resolve(__dirname, '..', '..', 'resources', 'atom.png') + icon: @iconPath message: 'There was an error checking for updates.' title: 'Update Error' detail: message