299897 - about:logo should display product specific logo on SeaMonkey instead of Mozilla star

Status: VERIFIED FIXED

(Firefox Build System :: General, defect, P1)

Description

[Robert Kaiser]

See bug 285696 comment 69 - I accidentally changed the about:logo image for
Camino along with SeaMonkey rebranding.

I guess the one you're using comes out of embed.jar, right? So it would be OK if
we would have our logo image in toolkit.jar for SeaMonkey (i.e. if MOZ_SUITE is
set) and else leave in the old Mozilla star image?

Comment 1

[Robert Kaiser]

Attachment: patch v1: include a SeaMonkey logo only #ifdef MOZ_SUITE

OK, I backed out the logo.gif for perf testing today and decided to not revert
the backout again, so at least it looks OK in Camino etc. for now.

Here's an a bit hacked solution for the problem:
This is including the SeaMonkey logo.gif in the xpfe toolkit.jar only #ifdef
MOZ_SUITE, else it's using the normal mozilla star logo.gif

Of course, the new logo.gif has to be checked in at the new location along with
that. I've been discussing the location with Neil today, the one in the patch
is not final yet, but it's easy to change.
It may be reasonable to use suite/base/branding/ instead of suite/app/

Comment 2

[Mike Pinkerton (not reading bugmail)]

i think bsmedberg wanted to get camino over to the xre sometime in the near
future, and that would involve pulling in toolkit, iirc.

Comment 3

[Mike Pinkerton (not reading bugmail)]

this really should be a core bug, i guess. moving it over there.

Comment 4

[Christian :Biesinger (don't email me, ping me on IRC)]

ok so, shouldn't branding-like things like this really be in chrome://branding/
somewhere?

Comment 5

[Benjamin Smedberg]

Yes, this sounds like it really does belong in chrome://branding/content/

Comment 6

[Robert Kaiser]

(In reply to comment #4)
> ok so, shouldn't branding-like things like this really be in chrome://branding/
> somewhere?

I believe that ideally, the whole about: and about:logo stuff should be there,
which would make it product-specific and make my hack unnecessary.

Unfortunately, this would need a patch across all products and toolkits, as
about:logo is hardwired to chrome://global/content/logo.gif in the global about
protocal code.

I'm morphing the bug description to the current state of this, as the Mozilla
star is back in and Camino should be fixed, but now SeaMonkey has to swallow the
problem of showing a very wrong about:logo on its about: page.

I believe my patch is still valid as a short-time hack to get it displayed
correctly.

It's a hack though.

Additionally, though I can't mark it, this bug is blocking-seamonkey1.0a+

Comment 7

[[e-mail]]

(In reply to comment #2)
>i think bsmedberg wanted to get camino over to the xre sometime in the near
>future, and that would involve pulling in toolkit, iirc.
Well toolkit doesn't have a logo at all.

Comment 8

[Robert Kaiser]

Attachment: patch v2: place an override into suite/ - not touching xpfe/global

OK, here's a patch that looks clear to me, though it still needs to override
toolkit.jar for SeaMonkey.

I'd like to get either the first or second patch in as a short-time fix, and
deal with a long-term fix, involving the about protocol handler, later.

Comment 9

[Robert Kaiser]

(In reply to comment #7)
> (In reply to comment #2)
> >i think bsmedberg wanted to get camino over to the xre sometime in the near
> >future, and that would involve pulling in toolkit, iirc.
> Well toolkit doesn't have a logo at all.

Just out of interest, how do they deal with about:logo being defined in the list
in
http://lxr.mozilla.org/mozilla/source/netwerk/protocol/about/src/nsAboutRedirector.cpp#60
?

Comment 10

[[e-mail]]

Comment on attachment 188532 [details] [diff] [review]
patch v1: include a SeaMonkey logo only #ifdef MOZ_SUITE

>+        content/global/logo.gif                     (/suite/app/logo.gif) 
r=me with a better path such as that discussed on IRC.

Comment 11

[Robert Kaiser]

Comment on attachment 188537 [details] [diff] [review]
patch v2: place an override into suite/ - not touching xpfe/global

In that case, this approach is obsolete.

The path we want to use for the logo now is /suite/chrome/branding/logo.gif (as
discussed on IRC)

Comment 12

[jag (Peter Annema)]

Comment on attachment 188532 [details] [diff] [review]
patch v1: include a SeaMonkey logo only #ifdef MOZ_SUITE

sr=me, provided the appropriate changes are made before you check in (and
perhaps attach the updated patch here?)

Comment 13

[Robert Kaiser]

Comment on attachment 188532 [details] [diff] [review]
patch v1: include a SeaMonkey logo only #ifdef MOZ_SUITE

requesting approval. This is xpfe-specific and only visible in SeaMonkey,
additionally blocking the SeaMonkey alpha.

Comment 14

[Robert Kaiser]

Comment on attachment 188532 [details] [diff] [review]
patch v1: include a SeaMonkey logo only #ifdef MOZ_SUITE

move a? to 1.8b4

Comment 15

[Robert Kaiser]

Checking in mozilla/suite/branding/content/logo.gif;
/cvsroot/mozilla/suite/branding/content/logo.gif,v  <--  logo.gif
initial revision: 1.1
done
Checking in mozilla/xpfe/global/jar.mn;
/cvsroot/mozilla/xpfe/global/jar.mn,v  <--  jar.mn
new revision: 1.85; previous revision: 1.84
done

This should be fixed now.

Comment 16

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b3) Gecko/20050714 SeaMonkey/1.0a]
(nightly) (W98SE)

V.Fixed.