1451713 - Use SVG icon for verification failed in update UI

Status: RESOLVED FIXED

(Toolkit :: Themes, enhancement)

Description

[Tim Nguyen :ntim]

Replacing:
https://dxr.mozilla.org/mozilla-central/source/toolkit/themes/windows/mozapps/update/updates.css#109
https://dxr.mozilla.org/mozilla-central/source/toolkit/themes/osx/mozapps/update/updates.css#133

with:

-moz-context-properties: fill;
fill: currentColor;
list-style-image: url(chrome://global/skin/icons/warning.svg);

allows removing 2 PNG icons:
toolkit/themes/windows/global/icons/information-16.png
toolkit/themes/osx/global/icons/notfound.png

Comment 1

[Manish [:manishkk]]

Attachment: Patch_Bug1451713

Please review

Comment 2

[Tim Nguyen :ntim]

Comment on attachment 8965296 [details] [diff] [review]
Patch_Bug1451713

Review of attachment 8965296 [details] [diff] [review]:
-----------------------------------------------------------------

You also need to remove those:

https://dxr.mozilla.org/mozilla-central/source/toolkit/themes/windows/global/jar.mn#49
https://dxr.mozilla.org/mozilla-central/source/toolkit/themes/osx/global/jar.mn#66

Comment 3

[Tim Nguyen :ntim]

Comment on attachment 8965296 [details] [diff] [review]
Patch_Bug1451713

Review of attachment 8965296 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/themes/windows/mozapps/update/updates.css
@@ +105,4 @@
>  }
>  
>  #verificationFailedIcon {
> + margin-left: 5px;

This indentation should remain unchanged.

Comment 4

[Manish [:manishkk]]

Attachment: PatchV2_Bug1451713

Please review

Comment 5

[Dão Gottwald [:dao]]

Comment on attachment 8965311 [details] [diff] [review]
PatchV2_Bug1451713

>--- a/toolkit/themes/osx/mozapps/update/updates.css
>+++ b/toolkit/themes/osx/mozapps/update/updates.css
>@@ -130,7 +130,10 @@ wizardpage {
> 
> #verificationFailedIcon {
>   margin-left: 5px;
>-  list-style-image: url("chrome://global/skin/icons/notfound.png");
>+  -moz-context-properties: fill;
>+  fill: currentColor;
>+  list-style-image: url(chrome://global/skin/icons/warning.svg);
>+

Please remove the empty line.

>--- a/toolkit/themes/windows/mozapps/update/updates.css
>+++ b/toolkit/themes/windows/mozapps/update/updates.css
>@@ -106,7 +106,10 @@ wizardpage {
> 
> #verificationFailedIcon {
>   margin-left: 5px;
>-  list-style-image: url("chrome://global/skin/icons/information-16.png");
>+  -moz-context-properties: fill;
>+  fill: currentColor;
>+  list-style-image: url(chrome://global/skin/icons/warning.svg);
>+

ditto

Thanks!

Comment 6

[Manish [:manishkk]]

:ntim

when I am applying this patch on my repo.
https://pastebin.mozilla.org/9082341

getting this error. while I roolback and revert all changes.

Comment 7

[Tim Nguyen :ntim]

(In reply to Manish Kumar from comment #6)
> :ntim
> 
> when I am applying this patch on my repo.
> https://pastebin.mozilla.org/9082341

This is because you already have the patch applied.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/878f06687e5a
Use SVG icon for verification failed in update UI. r=dao

Comment 9

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/878f06687e5a