Bug 1474132 - Choose right add-on icon size in about:addons and install popup draft
authorOriol Brufau <oriol-bugzilla@hotmail.com>
Fri, 13 Jul 2018 13:51:28 +0200
changeset 818012 0bd3c7b5a88cbdaf37209f9adbc1109338217b57
parent 817583 96c61b1dd0a1ecbc37fd4e257ff0bdff6f56df8b
push id116242
push userbmo:oriol-bugzilla@hotmail.com
push dateFri, 13 Jul 2018 20:28:55 +0000
bugs1474132
milestone63.0a1
Bug 1474132 - Choose right add-on icon size in about:addons and install popup MozReview-Commit-ID: L5O2OwlK879
browser/modules/ExtensionsUI.jsm
toolkit/mozapps/extensions/content/extensions.xml
--- a/browser/modules/ExtensionsUI.jsm
+++ b/browser/modules/ExtensionsUI.jsm
@@ -404,17 +404,17 @@ var ExtensionsUI = {
     return new Promise(resolve => {
       let action = {
         label: bundle.getString("addonPostInstall.okay.label"),
         accessKey: bundle.getString("addonPostInstall.okay.key"),
         callback: resolve,
       };
 
       let icon = addon.isWebExtension ?
-                 addon.iconURL || DEFAULT_EXTENSION_ICON :
+                 AddonManager.getPreferredIconURL(addon, 32, window) || DEFAULT_EXTENSION_ICON :
                  "chrome://browser/skin/addons/addon-install-installed.svg";
       let options = {
         hideClose: true,
         timeout: Date.now() + 30000,
         popupIconURL: icon,
         eventCallback(topic) {
           if (topic == "dismissed") {
             resolve();
--- a/toolkit/mozapps/extensions/content/extensions.xml
+++ b/toolkit/mozapps/extensions/content/extensions.xml
@@ -909,17 +909,17 @@
           this.mAddon = aAddon;
 
           this._installStatus.mAddon = this.mAddon;
           this._updateDates();
           this._updateState();
 
           this.setAttribute("name", aAddon.name);
 
-          var iconURL = AddonManager.getPreferredIconURL(aAddon, 48, window);
+          var iconURL = AddonManager.getPreferredIconURL(aAddon, 32, window);
           if (iconURL)
             this._icon.src = iconURL;
           else
             this._icon.src = "";
 
           if (this.mAddon.description)
             this._description.value = this.mAddon.description;
           else