1697806 - De-duplicate colored extension SVG icons

Status: RESOLVED DUPLICATE of bug 1708008

(Toolkit :: Themes, task, P3)

Description

[Tim Nguyen :ntim]

browser/components/extensions/extension.svg
toolkit/themes/shared/extensions/extensionGeneric.svg

are the same, but at different sizes.

Comment 1

[Tim Nguyen :ntim]

Attachment: Bug 1697806 - De-duplicate colored extension SVG icons. r=#extension-reviewers

Comment 2

[Luca Greco [:rpl] [:luca] [:lgreco]]

Hey Rob, would you mind to pick up the review for the attached patch?

(I kind of recall this and so I have the feeling that we may have another duplicate of this issue, I have the feeling it may have been a good first bug that was never completed, but I haven't checked and so I may be wrong about the duplicate, but if you find it let's close it as a duplicate of this one)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/18909220355a
De-duplicate colored extension SVG icons. r=robwu

Comment 4

[Rob Wu [:robwu]]

1421039(In reply to Luca Greco [:rpl] [:luca] [:lgreco] from comment #2)

(I kind of recall this and so I have the feeling that we may have another duplicate of this issue, I have the feeling it may have been a good first bug that was never completed, but I haven't checked and so I may be wrong about the duplicate, but if you find it let's close it as a duplicate of this one)

I think that you're thinking of bug 1421039

Comment 5

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/18909220355a

Comment 6

[Justin Peter]

The icon for a few custom extension I have (no icon included) just changed from green to gray, is that intended?

Comment 7

[Tim Nguyen :ntim]

(In reply to Justin Peter from comment #6)

The icon for a few custom extension I have (no icon included) just changed from green to gray, is that intended?

Nope, definitely not, could you please file a bug with some steps to reproduce/screenshot? Thanks!

Comment 8

[Justin Peter]

Alright, see bug 1698237.

Comment 9

[Tim Nguyen :ntim]

This is getting backed out because of bug 1698237 comment 4:
https://hg.mozilla.org/integration/autoland/rev/c3fdb26099a3

Rob, what do you think of this?

Probably easiest path is to remove the unpackaged colored extensionGeneric.svg in toolkit and (optionally) rename the browser extension.svg file to extension-colored.svg.

This is how categories-extensions.svg is packaged twice btw: https://searchfox.org/mozilla-central/rev/897104d3f309f1901190bcc8be87437ca3dab630/toolkit/themes/shared/mozapps.inc.mn#11,29

Would you mind taking this if you have time? I'm not sure what the expected end result regarding icons is (where to use the colored one, the monochrome category one, or the 16x16 extension one), it'd be faster if someone from your side takes this IMO.

Comment 10

[Justin Peter]

Obviously it's not my call, but I will note that the green icon did generally look a bit out-of-place in the UI. (I suppose the point is that any change be intentional, though.)

Comment 11

[Rob Wu [:robwu]]

An almost identical patch landed in bug 1708008.

The icon is still not green, but that has been deemed acceptable by the extension team. We mostly care that the icon looks different from the "Extensions" tab icon at about:addons (a hollow puzzle piece), which is the case (the default extension icon is a filled grey puzzle piece).