Opened 4 years ago

Last modified 14 months ago

#2057 new defect

High resolution icon for Chrome extension management page is missing

Reported by: sebastian Assignee:
Priority: P3 Milestone:
Module: Platform Keywords: externaldependency
Cc: sven, greiner, christiane, Ross Blocked By:
Blocking: Platform: Unknown
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description

Environment

Chrome and Opera, with high DPI configuration (e.g. Mac with Retina display)

How to reproduce

Go to chrome://extensions.

Observed behaviour

The 48px Adblock Plus icon is scaled up to 96px.

Expected behaviour

The extension should provide an icon at twice the size (i.e: 96px), to be used for high DPI configuration, as it does for all the other icon sizes.

Attachments (1)

icon_96px_detailed_v1_2.png (6.0 KB) - added by christiane 4 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 4 years ago by sebastian

@sven: Please attach a 96x96 version of the icon used on the extension management page.

comment:2 Changed 4 years ago by greiner

  • Priority changed from Unknown to P3

comment:3 Changed 4 years ago by sven

  • Cc christiane added

comment:4 Changed 4 years ago by sven

Adding christiane to this issue since she is the one creating the icon. Could you please create a 96x96px detailed icon for this purpose?

Changed 4 years ago by christiane

comment:5 Changed 4 years ago by sebastian

  • Keywords externaldependency added

Apparently Chrome is using the 48px icon there regardless. So at least for now, there is no point in adding a 96px icon to the build.

comment:6 Changed 18 months ago by Ross

  • Tester set to Unknown
  • Verified working unset

I'm not sure the status of this. There are icons up to 128px in the build and the manifest but I'm not sure which is being used for the chrome://extensions page (which seems to display the icon in 96px). The icon does look a little blurry, but that might be the source icons?

ABP 1.13.4
Chrome 61 / Windows 10

comment:7 Changed 18 months ago by sebastian

You can inspect the chrome://extensions page using the developer tools, copy the data: URL used as backround-image for the ABP logo and paste it into a new tab. Then when you inspect the image in the new tab, the developer tools should tell you which actual size that image has. (Alternatively, you could decode the base64 and analyse the image with external tools.)

If you are in a high-res setup (e.g. Mac Book with Retina display, or Windows 10 with scaling options in the display settings set to 200%), that image should be 96px (twice as on low-res). However, note that we currently don't provide an 96px icon, since, last time we checked, Chrome was ignoring high-res icons here, and always used the 48px icon.

If you have a high-res setup (I don't), you can check whether that is still the case, by creating a minimal extension containing following mainfest.json and two images in the respective size:

{ 
  "name": "icon-test",
  "version": "0.1",
  "manifest_version": 2,
  "icons": {
    "48": "icon48.png"
    "96": "icon96.png"
  }
}

If Chrome is picking up the 48px icon, no matter what, there is still nothing we can do here. But we might want to file a Chrome bug at least. However, if the 96px icon is picked up, we might want to add 96px icon to our builds as well.

comment:8 Changed 18 months ago by sebastian

  • Cc Ross added

comment:9 Changed 14 months ago by greiner

  • Component changed from User-Interface to Platform
Note: See TracTickets for help on using tickets.