1362210 - Update SeaMonkey branding icons

Status: RESOLVED FIXED

(SeaMonkey :: Themes, enhancement)

Description

[Ken Saunders]

Attachment: SeaMonkey-icns.png

User Agent: Mozilla/5.0 (Windows NT 6.1; rv:53.0) Gecko/20100101 Firefox/53.0
Build ID: 20170413192749




Expected results:

Per http://forums.mozillazine.org/viewtopic.php?p=14746198#p14746198
"It would be nice if SeaMonkey's icon in the Mac application package could be brought up to date. It is still 128x128"

barbaz
"feel free to contribute a 1024x1024 SeaMonkey icon for the Mac application package"


I don't have access to a Mac so I can't check the quality, etc. If I can be of further assistance, please let me know.

Zip contents see SeaMonkey-icns.png

Comment 1

[Ken Saunders]

Attachment: SeaMonkey-Mac-Icons.zip

Comment 2

[Frank-Rainer Grahl (:frg)]

IanN, I need to do installer changes because of bustage and also would like to ship hi-res icons with 2.57. What is needed so that we can use these icons here. Is an mpl V2 copyright statement in the zip all which is needed? They need to be converted to png for all platform use but are really nice.

Comment 3

[Ian Neal]

(In reply to Frank-Rainer Grahl (:frg) from comment #2)
> IanN, I need to do installer changes because of bustage and also would like
> to ship hi-res icons with 2.57. What is needed so that we can use these
> icons here. Is an mpl V2 copyright statement in the zip all which is needed?
> They need to be converted to png for all platform use but are really nice.

Maybe Robert knows this, otherwise might need to check with someone else.

Comment 4

[Stefan [:stefanh]]

(In reply to Frank-Rainer Grahl (:frg) from comment #2)
> They need to be converted to png for all platform use but are really nice.

For mac it should be the .icns format (which basically is a package of icons in different resolutions), see https://dxr.mozilla.org/comm-central/source/suite/branding/nightly/icons/mac

Comment 5

[Stefan [:stefanh]]

Ah, now when I looked in the .zip I see that it contains various .icns files.

Comment 6

[Ian Neal]

Hi, do you know what sort of copyright/licence statement is needed and where?

Comment 7

[Robert Kaiser]

MPL agreement should be fine - esp. if the artwork itself is icon material we already have and/or our Mozilla-trademarked logo. That said, we have high-quality SVG files esp. for the logo itself from which any resolution should be able to be derived from.

Comment 8

[Frank-Rainer Grahl (:frg)]

Attachment: 1362210-brandingicons-2538.patch

This add new higher resolution icons and svgs. The svgs and pngs were done by buc. I hope I got the latest version in. I moved our the original svg icons into a template folder and the new icons then into the former svg folder. There are currently not use but can be integrated in a follow-up.

Comment 9

[Dmitry Butskoy (:buc)]

Comment on attachment 9213496 [details] [diff] [review]
1362210-brandingicons-2538.patch

Forgot to add the main logo in svg form. The final v3 one is here https://bugzilla.mozilla.org/show_bug.cgi?id=1674589#c14 .

It could be "seamonkey.svg", "default.svg", "main-window.svg", or all the three together :)

Remove icons/template/svg/semonkey.svg. Unlike the old true "svg/*" stuff, it was just our temporary attempt.

Some optional notes:

It looks that icons/gtk/mainWindow*.png are a bit out of order with another ones there. Could be just main-window.png (32x32), main-window16.png and main-window48.png . The other sizes are present as "default*.png" anyway.

The "default512.png" and "default1024.png" could be dropped. These sizes required for .icns format, but looks extra in a standalone form. (Any system where such sizes could be needed are moderm enough to support svg instead).

Comment 10

[Ian Neal]

Comment on attachment 9213496 [details] [diff] [review]
1362210-brandingicons-2538.patch

>+++ b/suite/branding/branding-common.mozbuild

         if CONFIG['MOZ_WIDGET_TOOLKIT'] == 'windows':
>+            icon_suffix = '.ico'
>+            icon_dir = CONFIG['MOZ_WIDGET_TOOLKIT']
Doesn't this just overcomplicate things, surely icon_dir = 'windows' is clearer?

Firefox / TB don't seem to have the equivalent of mainwindow* images just default*, do we need them still?

Comment 11

[Frank-Rainer Grahl (:frg)]

Attachment: 1362210-brandingicons-v1_2-2538.patch

New version with basically default*.png main-window*.png and icon*.png duplicated.

Comment 12

[Frank-Rainer Grahl (:frg)]

Attachment: 1362210-brandingicons-2538.patch

I think this is reviewable. Removed the main-window* icons and didn't spot a difference under Linux. Default*.png seems to be used as a fallback. seamonkey-v3.svg is in and removed under templates.

Next stage would be to add the svg(s) into the package.

FRG

Comment 13

[Frank-Rainer Grahl (:frg)]

Attachment: 1362210-brandingicons-v1_3-2538.patch

It helps if you pick the right patch...

Comment 14

[Frank-Rainer Grahl (:frg)]

Attachment: 1362210-brandingicons-v1_3-2538.patch

Yes the package-manifest.in was bad. Bill would have found out for the Linux build. doh

Comment 15

[Dmitry Butskoy (:buc)]

Comment on attachment 9214998 [details] [diff] [review]
1362210-brandingicons-v1_3-2538.patch

LGTM

Comment 16

[Frank-Rainer Grahl (:frg)]

Comment on attachment 9214998 [details] [diff] [review]
1362210-brandingicons-v1_3-2538.patch

[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: blurry branding icons
Testing completed (on m-c, etc.): 2.53.8b1 pre
Risk to taking this patch (and alternatives if risky): not much of a bug.
String changes made by this patch: --

Comment 17

[Ian Neal]

Comment on attachment 9214998 [details] [diff] [review]
1362210-brandingicons-v1_3-2538.patch

>+++ b/suite/branding/branding-common.mozbuild
>+            FINAL_TARGET_FILES.chrome.icons.default += [
>+                'default128.png',
>+                'default16.png',
>+                'default22.png',
>+                'default24.png',
>+                'default256.png',
>+                'default32.png',
>+                'default48.png',
>+                'default64.png',
I'd prefer this sorted by number so 16, 22, 24, 32, 48, 64, 128, 256


>+++ b/suite/installer/package-manifest.in
>@@ -587,21 +587,24 @@
> ; shell icons
> #ifdef MOZ_GTK
> @RESPATH@/chrome/icons/default/bookmarkproperties.png
> @RESPATH@/chrome/icons/default/bookmarkproperties16.png
> @RESPATH@/chrome/icons/default/bookmarkproperties48.png
> @RESPATH@/chrome/icons/default/chatzilla-window.png
> @RESPATH@/chrome/icons/default/chatzilla-window16.png
> @RESPATH@/chrome/icons/default/chatzilla-window48.png
>-@RESPATH@/chrome/icons/default/default.png
>+@RESPATH@/chrome/icons/default/default128.png
> @RESPATH@/chrome/icons/default/default16.png
>+@RESPATH@/chrome/icons/default/default22.png
>+@RESPATH@/chrome/icons/default/default24.png
>+@RESPATH@/chrome/icons/default/default256.png
>+@RESPATH@/chrome/icons/default/default32.png
> @RESPATH@/chrome/icons/default/default48.png
> @RESPATH@/chrome/icons/default/default64.png
>-@RESPATH@/chrome/icons/default/default128.png
I'd prefer this sorted by number so 16, 22, 24, 32, 48, 64, 128, 256

r/a=me with those addressed.

Comment 18

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/c81bf129cf44
Update SeaMonkey branding and OS integration icons. r=frg

Comment 19

[Frank-Rainer Grahl (:frg)]

Landed with the wrong reviewer doh.

The order in the suite/branding/branding-common.mozbuild couldn't be changed. The build system expects a sorted list here and unfortunately that means 128 and 256 are in the middle of it. I didn't think splitting it was worth it as discussed on irc. package-manifest.in changed for checkin.

Comment 20

[Bob Edenhofer]

For being totally blind, I really enjoy reading all these patches, just out of curiosity, I wonder, as to what they mean, I don't want to make anyone mad, for wanting to learn new things.

Comment 21

[Frank-Rainer Grahl (:frg)]

https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/508ea3afc611feb704980c41e58b5c1bb30d44fe
Update SeaMonkey branding and OS integration icons. r=IanN a=IanN