375080 - Firefox logo is not the official one

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect)

Description

[Pascal Chevrel:pascalc (PTO until April 26)]

On https://addons.mozilla.org/fr/firefox/ the firefox logo does not respect our visual identity guidelines (http://www.mozilla.org/foundation/identity-guidelines/firefox.html)

Ragged edges, no logo, lacks the copyright symbol.

Comment 1

[Cameron]

The logos on http://en-us.www.mozilla.com/en-US/firefox/ and
http://en-us.www.mozilla.com/en-US/firefox/all.html don't include the copyright
symbol either.. so I've filed bug 375295 against mozilla.com to find out how
important it is to follow these guidelines. Putting a little copyright symbol
there is going to look a bit ugly I think.

Comment 2

[Pascal Chevrel:pascalc (PTO until April 26)]

actually I am more worried about the lack of shadow, it makes the logo a bit rough ;)

Comment 3

[Paul Kim]

See bug 375295 for more discussion -- as Cameron notes there we don't follow the copyright symbol rule consistently on mozilla.com or mozilla-europe.org either. After reviewing the Firefox logo on AMO it does look a bit rough around the edges. Cameron -- I'm going to email you a high res copy of the logo to play with that contains the shadow. Steven Garrity can also advise you on optimizing the logo for best display at the size you're using on AMO.

Comment 4

[Paul Kim]

Let me also add that based on the usage of the Firefox logo elsewhere on mozilla.com IMO you don't need to use the copyright symbol in this instance. I'll clarify what the plan is for updating the visual identity guidelines, and what the legal requirements are for displaying the copyright symbol.

Comment 5

[Cameron]

Attachment: New Firefox logo

At this size, you can't even see the copyright symbol anyway, although the Thunderbird one seems to have it... https://addons.mozilla.org/en-US/thunderbird

Requesting review for the sake of protocol.

Comment 6

[Steven Garrity [:sgarrity]]

(In reply to comment #5)
> Requesting review for the sake of protocol.

Cameron - this looks good, but is a full-color alpha-transparent PNG, which will have a white background in Internet Explorer. We'll probably have to convert the image to 256-color to have 1-bit transparency for the sake of IE.

Since this image lives on top of a gradient backgorund, this will have to be done with some care to make sure it still blends nicely into the background.

Cameron, is this something you can have a go at? If not, let me know and I'll do it.

Comment 7

[Cameron]

Attachment: 256 colour png

The images for the other applications are full colour alpha transparent PNGs... and since IE7 supports this and any user with Windows XP should have received an automatic update notification regarding it, and we're making a site targeted at Mozilla-based browsers, the amount I care about IE6 users is rapidly approaching 0.

That said - is this what you wanted? I've not experience doing this before. If I look closely enough I think I can still see a bit of a ring there. Screenshot shortly.

Comment 8

[Cameron]

Attachment: Screenshot of 256 colour PNG over gradient background

Comment 9

[Steven Garrity [:sgarrity]]

Cameron, since this logo is used over top of other backgrounds on the site, let's keep the full alpha-transparent version you had already (sorry for the extra work).

Comment 10

[Steven Garrity [:sgarrity]]

Cameron, do you have what you need to commit this?

Comment 11

[Cameron]

(In reply to comment #10)
> Cameron, do you have what you need to commit this?
> 

I was waiting for pkim's r+ with regard to the copyright symbol.

Comment 12

[Cameron]

Checked in.

SVN r3073

This won't hit production until the next batch of updates goes through.

Comment 13

[Paul Kim]

Sorry for the lag Cameron - just filed a bug to sync mozilla.com to grab this change.

Comment 14

[Reed Loden [:reed]]

(In reply to comment #13)
> Sorry for the lag Cameron - just filed a bug to sync mozilla.com to grab this
> change.

This is for AMO, not mozilla.com. morgamic / clouserw will need to tag this and request a sync themselves.

Comment 15

[Paul Kim]

My bad -- morgamic: can you review?

Comment 16

[Michael Morgan [:morgamic]]

Sure -- I think this looks good.  +r=morgamic.  This was apparently already checked in by Cameron... ?

Anyway, we'll have to merge again to the production tag and check w/ the other guys in the morning to see if we're ready to push the next batch.

Will update in the AM.

Comment 17

[Michael Morgan [:morgamic]]

Looks great after the update.

Comment 18

[Paul Kim]

+1. shiny. thanks cameron!