119765 - Modern theme description talks about Communicator

Status: RESOLVED FIXED

(SeaMonkey :: Themes, defect)

Description

[Henrik Gemal]

Go into Edit -> Prefs -> Apperance -> Themes and mark Modern

The text talks about "Communicator". But there's nothing called Communicator...!
The text should be updated.

build 20020111

Comment 1

[Stephen Walker]

*** Bug 126959 has been marked as a duplicate of this bug. ***

Comment 2

[Greg Valure]

Easy enough to fix...

Comment 3

[Greg Valure]

-> me

Comment 4

[Greg Valure]

Attachment: Patch

Comment 5

[Greg Valure]

Attachment: Patch (neutral language)

On second thought, maybe it would be better if we didn't mention any specific
product at all.  This way we won't need separate versions for Netscape and
Mozilla.

Comment 6

[Greg Valure]

Back to default owner.  Sorry.

I don't know why I can't get a reviewer response on this.  Before they were on
vacation, but now I don't know what's up.  If someone else wants to try to get
this reviewed and checked in, please do.

Comment 7

[Ben Bucksch (:BenB)]

This bug is invalid.

> But there's nothing called Communicator...!

There is. The Navigator/Mailnews/Composer appsuite is technically called
Communicator, see source xpfe/communicator/.

Comment 8

[Greg Valure]

Technically.  But there's nothing exposed to the user referencing Communicator;
nothing in the UI, nothing in the help docs.  Even the Netscape-branded version
is now just Netscape and not Communicator.  We should not expose something that
exists as a source-level convention only to the user.

Comment 9

[Andreas Kunz]

Comment on attachment 94541 [details] [diff] [review]
Patch (neutral language)

Requesting review from [e-mail] for Gregs patch - see comment #6.
I pondered creating a patch with the app name in it, but I think it is better
this way.

Comment 10

[shliang]

Comment on attachment 94541 [details] [diff] [review]
Patch (neutral language)

I don't think we shoud change this. As mentioned in comment #5, Communicator
does refer to something.

Comment 11

[Andreas Kunz]

shliang, so this is WONTFIX?

I rather think this should be fixed because "Communicator" has no real meaning
in the Mozilla world. But I don't mind if it isn't.
The main thing is this bug gets resolved one or the other way.

Comment 12

[shliang]

marking invalid

Comment 13

[Henrik Gemal]

come on...

Just try to think as a user sometimes! Refering to some technical stuff is just
plain stupid. This is nothing called Communicator from a users point of view!
When downloading, installing and using Mozilla it's always called Navigator!

Comment 14

[Brendan Eich [:brendan]]

Communicator may be used in Mozilla code ("Mozilla Communicator", e.g.), but if
it is in fact not being used, why start now?

/be

Comment 15

[Galileo Vidoni]

but... remember that, for an M$ assimilated user, "Mozilla" isn't more than a
name similar do Godzilla. It isn't suggestive to internet. Not as "Internet
Explorer". Restoring the "Communicator" world (I know that in firebird this
isn't possible, but perhaps when there appear releases of Mozilla containing
Fbird and tbird, and standalone composer), would be good as it sound to
communicate, at least...

Comment 16

[Daniel Wang]

Comment on attachment 94541 [details] [diff] [review]
Patch (neutral language)

"Communicator" should never appear on Mozilla UI. I don't see any good reason
why this bug should not be fixed.

Comment 17

[Daniel Wang]

Neil, I don't have CVS account. can you check this in?

Comment 18

[Stefan Borggraefe]

Comment on attachment 94541 [details] [diff] [review]
Patch (neutral language)

be: can I take your comment 14 in this bug as an sr+? ;-) Thanks!

I can check this in then.

Comment 19

[Brendan Eich [:brendan]]

Comment on attachment 94541 [details] [diff] [review]
Patch (neutral language)

Sure.  If neil wants to r+sr these sorts of changes, that's ok with me.

/be

Comment 20

[Stefan Borggraefe]

Fix checked in.