1050809 - [10.10] Update toolbar and panel icons for OS X Yosemite

Status: VERIFIED FIXED

(Firefox :: Theme, defect)

Description

[:Gijs (he/him)]

Mockup and assets should be in bug 1020551. Please also doublecheck that they've been compressed fully with ImageOptim or similar.

This will probably need to be an OS-version-specific override, because the difference with the Lion-like icons we ship right now is quite stark, so we can't just replace the existing ones with Yosemite ones everywhere.

Comment 1

[Liz Henry (:lizzard) (relman/hg->git project)]

Juan, do you want to take Yosemite verifications ? If not let me know.

Comment 2

[juan becerra [:juanb]]

I don't have a working Yosemite installation, but I will have something to test on next week.

Comment 3

[:Gijs (he/him)]

Michael, this seems to be missing icons for Loop, as well as the small menuPanel icons (cut/copy/paste, zoom buttons - see menuPanel-small.png) and perhaps the footer icons for the panel? The new tab button, all tabs button (dropdown arrow) and tab navigation arrows are also still the same (possibly also less important).

I think we should at least do the menuPanel-small.png update in this bug, because otherwise the menu looks really wonky. Everything else could be followup if you prefer? Let me know what you think.

Comment 4

[:Gijs (he/him)]

Attachment: update toolbar/menupanel icons for yosemite,

Pretty simple approach, same as for the lion icons - Jared, does this look right to you? I'd want to do the same for menuPanel-small.png and anything else we'd still update this round.

Comment 5

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

Comment on attachment 8478629 [details] [diff] [review]
update toolbar/menupanel icons for yosemite,

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

Unable to test this, but this looks like the right approach.

Comment 6

[:Gijs (he/him)]

I believe this also misses out the in-urlbar icons (stop/reload/go).

Comment 7

[Michael Maslaney [:mmaslaney] ([e-mail])]

Attachment: Yosemite-assets.zip

Updated assets.

Comment 8

[:Gijs (he/him)]

Attachment: update toolbar/menupanel icons for yosemite,

Now with a gazillion more icons.

Comment 9

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

Comment on attachment 8480674 [details] [diff] [review]
update toolbar/menupanel icons for yosemite,

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

We really should get a bug on file to merge more of these files in to a single sprite. It is crazy how many various files there are to find and update (the @2x makes it seem worse).

f+ because the Sync icon in the menu panel is mismatched, http://screencast.com/t/XBuq2LcwpK

Comment 10

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

For comparison, http://screencast.com/t/QeKgdVnnfN04

Comment 11

[:Gijs (he/him)]

Michael, could you provide the sync icon as well? I sure hope this is the last one... :-(

Comment 12

[Dão Gottwald [:dao]]

Comment on attachment 8480674 [details] [diff] [review]
update toolbar/menupanel icons for yosemite,

>diff --git a/browser/themes/osx/yosemite/Toolbar.png b/browser/themes/osx/yosemite/Toolbar.png

This is inconsitent with where e.g. lion and aero icons live in the source tree. They use file name suffixes rather than separate folders. I think the former makes more sense, as it keeps the various versions of the same image together. I don't see much value in having all yosemite icons in one folder.

Comment 13

[Michael Maslaney [:mmaslaney] ([e-mail])]

Attachment: sync-yosemite.zip

Sync assets attached

Comment 14

[:Gijs (he/him)]

(In reply to Dão Gottwald [:dao] from comment #12)
> Comment on attachment 8480674 [details] [diff] [review]
> update toolbar/menupanel icons for yosemite,
> 
> >diff --git a/browser/themes/osx/yosemite/Toolbar.png b/browser/themes/osx/yosemite/Toolbar.png
> 
> This is inconsitent with where e.g. lion and aero icons live in the source
> tree. They use file name suffixes rather than separate folders. I think the
> former makes more sense, as it keeps the various versions of the same image
> together. I don't see much value in having all yosemite icons in one folder.

... whereas I thought the root of that folder was already cluttered enough, and duplicating a boatload of images and their @2x equivalents wouldn't make anyone happier.

Comment 15

[:Gijs (he/him)]

Attachment: update toolbar/menupanel icons for yosemite,

Now with sync and with '-yosemite' filenames instead of a directory (still using a directory inside the jar because that's also what lion and aero do).

Comment 16

[Dão Gottwald [:dao]]

(In reply to :Gijs Kruitbosch from comment #14)
> (In reply to Dão Gottwald [:dao] from comment #12)
> > Comment on attachment 8480674 [details] [diff] [review]
> > update toolbar/menupanel icons for yosemite,
> > 
> > >diff --git a/browser/themes/osx/yosemite/Toolbar.png b/browser/themes/osx/yosemite/Toolbar.png
> > 
> > This is inconsitent with where e.g. lion and aero icons live in the source
> > tree. They use file name suffixes rather than separate folders. I think the
> > former makes more sense, as it keeps the various versions of the same image
> > together. I don't see much value in having all yosemite icons in one folder.
> 
> ... whereas I thought the root of that folder was already cluttered enough,

I'm not sure this is a problem in terms of maintainability, but if so we should break up the folder into more sub folders such as "newtab", "tabbrowser" etc.

> and duplicating a boatload of images and their @2x equivalents wouldn't make
> anyone happier.

It makes it more obvious what different versions exist of each image, which might come handy when updating images. The duplication exists either way, hiding it doesn't seem helpful.

Comment 17

[:Gijs (he/him)]

Comment on attachment 8480914 [details] [diff] [review]
update toolbar/menupanel icons for yosemite,

Michael, Jared kindly pointed out to me the customize exit button is a close icon instead of the shutdown icon - see

data:text/html,<body style="background-color: blue;"><img src="http://mxr.mozilla.org/mozilla-central/source/browser/themes/osx/menuPanel-exit.png?raw=1">

for the current one. Can you provide updated versions of this icon? Thanks!

Comment 18

[Michael Maslaney [:mmaslaney] ([e-mail])]

Attachment: exit.zip

Exit button attached.

Comment 19

[:Gijs (he/him)]

Attachment: update toolbar/menupanel icons for yosemite,

Now with new shutdown goodness.

Comment 20

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

Comment on attachment 8481416 [details] [diff] [review]
update toolbar/menupanel icons for yosemite,

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

Looks good. Thanks for the quick turn-around-time.

Comment 21

[:Gijs (he/him)]

(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #20)
> Comment on attachment 8481416 [details] [diff] [review]
> update toolbar/menupanel icons for yosemite,
> 
> Review of attachment 8481416 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Looks good. Thanks for the quick turn-around-time.

Likewise, thanks for the quick review, and sorry it took so many tries to get right. :-\

remote:   https://hg.mozilla.org/integration/fx-team/rev/c98e45bed5e0

Comment 22

[Ryan VanderMeulen [:RyanVM]]

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

Comment 23

[Tim Nguyen :ntim]

In the ZIP file, I also see that the help button misses an inverted icon. It might have been fixed in the patch though.

Comment 24

[Tim Nguyen :ntim]

Filed bug 1060944 for the issue.

Comment 25

[Catalin Varga [QA][:VarCat]]

Verified as fixed using the following environment:
FF 34
Build id: 20141009004002
OS: Mac Os X 10.10