426171 - Icon and preview in the addons manager for the default theme

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[Michael Ventnor]

Got some wonderful icons from one of the Tango guys on IRC a few nights ago. This is to replace the icon and preview in the Theme section of the addons manager since those still use Firefox 2 artifacts.

Comment 1

[Michael Ventnor]

Attachment: preview.png

Follows pretty much the same layout except more colourful. We can't use the standard toolbar icons like before since those are now dynamic on Linux.

Comment 2

[Michael Ventnor]

Attachment: icon.png

We can't use the reload icon here like before for aforementioned reasons, plus because the "reloaded" pun was Firefox 2's. So we took the folded shirt metaphor that Linux now uses for themes.

Take a look at it and see if you can guess the inspiration for the style and colours. Ingenious, no? :-)

Also, reed, what's the process for image-only checkins? Just approval is necessary, IIRC?

Comment 3

[Reed Loden [:reed]]

While we don't need to completely follow http://www.mozilla.org/foundation/identity-guidelines/firefox.html, I do think we probably need the (R) for the word "Firefox"... gerv or David, could you let us know what this needs to have?

Comment 4

[David Boswell]

Copying Frank for his feedback.

Comment 5

[Frank Hecker]

(In reply to comment #3)
> While we don't need to completely follow
> http://www.mozilla.org/foundation/identity-guidelines/firefox.html, I do think
> we probably need the (R) for the word "Firefox"... gerv or David, could you let
> us know what this needs to have?

Yes, it would be good practice to use the "R in circle" symbol for the word "Firefox", since it is in fact a registered trademark and we do this for other uses of the term (e.g., on the mozilla.com home page).

Comment 6

[Michael Ventnor]

The only reason I didn't bother was because the Firefox 2 preview didn't have the (R).

Comment 7

[Reed Loden [:reed]]

(In reply to comment #6)
> The only reason I didn't bother was because the Firefox 2 preview didn't have
> the (R).

Yep, that was a mistake. I've asked faaborg to make sure the new preview icons for Firefox 3 on all OSs include the (R).

Comment 8

[Michael Ventnor]

Attachment: preview.png (R)

Comment 9

[Reed Loden [:reed]]

Comment on attachment 312861 [details]
preview.png (R)

This doesn't match the real logo... the (R) is the wrong color and is in the wrong spot.

Comment 10

[Michael Ventnor]

Attachment: preview.png (R) 2

I didn't know you were a ui-reviewer reed ;) Shall I ask you again? :P

Also, I need to know where the theme version and description is kept, I can't use MXR because the search takes so long that the server kills it before its finished, leaving me with nothing.

Comment 11

[Reed Loden [:reed]]

(In reply to comment #10)
> I didn't know you were a ui-reviewer reed ;) Shall I ask you again? :P

Minuses are always free. ;)

This doesn't look like the right gray, as compared to what http://www.mozilla.com/img/firefox-title.png is using. Looks like you need to use #545454 (RGB: 84, 84, 84).

Change that, and then request ui-r from faaborg, please. :)

> Also, I need to know where the theme version and description is kept, I can't
> use MXR because the search takes so long that the server kills it before its
> finished, leaving me with nothing.

Already handling this in bug 423177.

Comment 12

[Michael Ventnor]

Comment on attachment 312713 [details]
icon.png

I fail to see why a ui-r from faaborg is necessary.

Comment 13

[Reed Loden [:reed]]

Has faaborg ui-reviewed these icons yet? As far as I know, he reviews all the icons the Tango team submits before they are attached to a bug for committing. If he hasn't, he needs to, as it's his call.

Comment 14

[Michael Ventnor]

Attachment: Right colour of the (R)

I really wonder who is going to notice a different hue on the (R) unless they're explicitly looking for it?

Comment 15

[Alex Faaborg [:faaborg] (Firefox UX)]

Attachment: Style guidelines

>I fail to see why a ui-r from faaborg is necessary.

Because we are crazy OCD about our word mark, and I have access to detailed documents chock full of rules :)

The three things we need to get right are:
-Font is meta bold roman
-Color should be rgb(214, 66, 3) #D64203 
-(r) mark should be black, and needs to be aligned to the top of the x

Comment 16

[Alex Faaborg [:faaborg] (Firefox UX)]

Comment on attachment 312868 [details]
Right colour of the (R)

I need to check on if we are allowed to use different colors for the word (for instance our new home page uses a darker red).  In the meantime, the aliasing on the text seems wrong, particularly on either side of the f.  Do you have the font or are you resizing an existing image?  I can send you the font if you need it.

Comment 17

[Alex Faaborg [:faaborg] (Firefox UX)]

Looking at the style guidelines I guess the (R) is supposed to match the word mozilla if it is present, and would be in #666666 or rgb(102,102,102)

Here are the guidelines in case anyone is curious:
http://people.mozilla.com/~faaborg/files/granParadisoUI/icons/mozstyle.pdf

Comment 18

[Michael Ventnor]

Alex, we used an existing image. Sending me the font would be appreciated, just send it as an attachment to my bugmail.

Comment 19

[Michael Ventnor]

Attachment: preview.png - Hopefully right this time

faaborg, can you please also ui-r the icon.png? (And hurry? RC1 is getting close)

Comment 20

[Alex Faaborg [:faaborg] (Firefox UX)]

Comment on attachment 313587 [details]
preview.png - Hopefully right this time

Looks great

Comment 21

[Alex Faaborg [:faaborg] (Firefox UX)]

Comment on attachment 312713 [details]
icon.png

Looks fine, bonus points for using a tango style firefox logo on the blue shirt instead of a stripe :)

Comment 22

[Alex Faaborg [:faaborg] (Firefox UX)]

>Take a look at it and see if you can guess the inspiration for the style and
>colours. Ingenious, no? :-)

I'm at a loss, what was the inspiration?

Comment 23

[Michael Ventnor]

(In reply to comment #21)
> (From update of attachment 312713 [details])
> Looks fine, bonus points for using a tango style firefox logo on the blue shirt
> instead of a stripe :)
> 

I didn't do so because I thought the icon was going to change soon? (Plus, subtlety is what makes it more memorable :) )

(In reply to comment #22)
> >Take a look at it and see if you can guess the inspiration for the style and
> >colours. Ingenious, no? :-)
> 
> I'm at a loss, what was the inspiration?
> 

What other graphic do you know of that uses those colours? :)

Comment 24

[Reed Loden [:reed]]

Checking in browser/themes/gnomestripe/browser/icon.png;
/cvsroot/mozilla/browser/themes/gnomestripe/browser/icon.png,v  <--  icon.png
new revision: 1.5; previous revision: 1.4
done
Checking in browser/themes/gnomestripe/browser/preview.png;
/cvsroot/mozilla/browser/themes/gnomestripe/browser/preview.png,v  <--  preview.png
new revision: 1.5; previous revision: 1.4
done