1347928 - Remove legacy information-*.png icons

Status: RESOLVED FIXED

(Toolkit :: Themes, enhancement)

Description

[Dão Gottwald [:dao]]

I'm going to attach a patch that uses chrome://global/skin/icons/info.svg and chrome://browser/skin/info.svg where appropriate and removes unused information-*.png files. Some of them were already unused without my changes. This will only leave behind information-16.png on Windows.

Comment 1

[Florian Quèze [:florian]]

The currently unused information-* files are:
chrome://global/skin/icons/information-32.png unused on linux
chrome://global/skin/icons/information-large.png unused on mac

Comment 2

[Dão Gottwald [:dao]]

Attachment: Bug 1347928 - Remove legacy information-*.png icons.

Review commit: https://reviewboard.mozilla.org/r/121060/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/121060/

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8848083 [details]
Bug 1347928 - Remove legacy information-*.png icons.

https://reviewboard.mozilla.org/r/121060/#review123924

::: toolkit/themes/osx/global/global.css:77
(Diff revision 1)
>  
>  .message-icon {
> -  list-style-image: url("chrome://global/skin/icons/information-64.png");
> +  list-style-image: url("chrome://global/skin/icons/info.svg");
>  }
>  
>  .alert-dialog #info\.icon,

Can you file a good-first-bug to rename these #info.* IDs to #info-* ?

It looks like they were introduced by bug 213228, http://searchfox.org/mozilla-central/diff/08a739f6d183178861de18a8619a1fa15f17628b/toolkit/content/commonDialog.xul#25

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d9839e96fe10
Remove legacy information-*.png icons. r=jaws

Comment 5

[Dão Gottwald [:dao]]

(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #3)
> Can you file a good-first-bug to rename these #info.* IDs to #info-* ?

This seems like a nonstarter since this stuff is used by add-ons.

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/d9839e96fe10