Open Bug 1623405 Opened 4 years ago Updated 3 years ago

MDN and about:addons inconsistence about extension icons

Categories

(Toolkit :: Add-ons Manager, enhancement, P3)

enhancement

Tracking

()

UNCONFIRMED

People

(Reporter: remtanmajitenshi, Unassigned)

Details

(Keywords: dev-doc-needed)

MDN article https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/icons says:

You should supply at least a main extension icon, ideally 48x48 px in size. This is the default icon that will be used in the Add-ons Manager.

But in current about:addons it seems that 32x32 px icons are used. This results in many extensions icons to download from addons.cdn.mozilla.net and fail without internet. Including Mozilla's own https://addons.mozilla.org/en-US/firefox/addon/facebook-container/ and https://addons.mozilla.org/en-US/firefox/addon/multi-account-containers/, which have only 48 and 96 px icons in manifest, but not 32 px.
See also: https://github.com/uBlockOrigin/uBlock-issues/issues/939#issuecomment-597081070

Either use 48x48 in about:addons (more compatible with existing extensions) or update MDN article to current state and inform developers about change.

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Product: Firefox → WebExtensions

I'm assuming that the Frontend component is correct since this enhancement is referring to the icons of webextensions.

Hopefully, this is an error in the specs so let's leave this to the guys at Webextensions. NI me if manual testing is needed.

Type: defect → enhancement
Component: Untriaged → Frontend
Flags: needinfo?(remtanmajitenshi)
Flags: needinfo?(remtanmajitenshi) → needinfo?(mixedpuppy)

Mark, is this a known issue?

Flags: needinfo?(mixedpuppy) → needinfo?(mstriemer)
Component: Frontend → Add-ons Manager
Product: WebExtensions → Toolkit

48x48 is used for hi dpi.

aboutaddons.css does define 32px. We should update documentation.

Severity: normal → S4
Keywords: dev-doc-needed
Priority: -- → P3
Flags: needinfo?(mstriemer)
You need to log in before you can comment on or make changes to this bug.