483360 - config.css/netError.css need a transparent Camino logo

Status: RESOLVED FIXED

(Camino Graveyard :: General, defect)

Description

[philippe (part-time)]

Currently the images have a white opaque background.
This affects people who set their own colors in Preferences > Appearance and check the 'Always use' box. About:config and netError.xhtml look odd in that case.

Sam, do you have the original artwork for that (Camino logo with alert icon) ? PSD @ 100px by 100px

Comment 1

[philippe (part-time)]

Oops, how did I get the browser in that state ? 
Preferences > Appearance, 'always us my colors' suppresses the images in those About:config and netError.xhtml files.

Still the point stands, in case the background-color in the stylesheet changes.

Comment 2

[Samuel Sidler (old account; do not CC)]

Hm, looks like I don't have a PSD for that artwork, only a gif and (non-transparent) png. I can probably recreate this later.

Comment 3

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

I forgot about this bug with last week's icon work, but we should still fix warning.png at some point.

I believe Jon used the now-removed alert-exclam.gif for the badge (or something close; the badge has more of a shadow underneath than the original does): http://mxr.mozilla.org/mozilla1.8/source/camino/embed-replacements/skin/classic/global/icons/alert-exclam.gif

Comment 4

[philippe (part-time)]

Attachment: warning.png transparent (WIP)

(In reply to comment #3)

> I believe Jon used the now-removed alert-exclam.gif for the badge (or something
> close; the badge has more of a shadow underneath than the original does):
> http://mxr.mozilla.org/mozilla1.8/source/camino/embed-replacements/skin/classic/global/icons/alert-exclam.gif

If you open that image in Preview (assuming the default grey background), you'll see the problem to reuse it: the GIF transparency pixels. If nothing else exist, lets try to rebuild something based on that.

Attached is a wip based on that image. I mostly erased the border & shadow and only recreated those parts. It is not a 100% match, though; the rounding in the bottom corners is a bit different, and the shadow is not an exact replica...
I think I did manage to erase/hide the gif compression artifacts.

I then overlayed it over the Camino icon I used for the blacklist.png image.

Let me know what you think.

Comment 5

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 390638 [details]
warning.png transparent (WIP)

This looks OK; what I noticed the most was that the left (and right, to a lesser extent) edges are not as strong/black as before.  The left seems like it's disappearing into the logo.  Can that be tweaked?

I did notice the corners/shadow, but they're not very noticeable; I'm more concerned with the sides.

Now that we're using the same base image, we can simplify the CSS and have both logos in exactly the same place; patch for that forthcoming.

Comment 6

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: netError.css unification/simplification

This makes the warning icon use the same position as the blocked icon and consolidates the rules.

Comment 7

[philippe (part-time)]

Comment on attachment 390682 [details] [diff] [review]
netError.css unification/simplification

r = me for the css changes.

I'll work on a more polished image tomorrow.

Comment 8

[philippe (part-time)]

Attachment: warning.png v2

This ought to do it.
Better rounding in the bottom corners, better edge at the top.

Comment 9

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 390760 [details]
warning.png v2

I think it might look even better than the original, at least for the sides ;) r=ardissone

Comment 10

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 390682 [details] [diff] [review]
netError.css unification/simplification

This just normalizes and consolidates the CSS now that both images have the same base Camino logo.

Comment 11

[Stuart Morgan]

Comment on attachment 390682 [details] [diff] [review]
netError.css unification/simplification

sr=smorgan

Comment 12

[Stuart Morgan]

Comment on attachment 390760 [details]
warning.png v2

sr=smorgan

Comment 13

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Checked in on cvs trunk and the CAMINO_2_0_BRANCH.