521647 - "Help" item in the menu bar should be different between Windows and other platforms for some locales

Status: VERIFIED FIXED

(SeaMonkey :: OS Integration, defect)

Description

[Cédric Corazza]

Attachment: Patch for Help item menu

For the french localisation we need to have a different "Help" menu between
Windows and Unix/Mac.
In French Windows, all the programs have an "?" item menu instead of "Help". Other French localized Mozilla products use "?" for the Help item (cf. https://bugzilla.mozilla.org/show_bug.cgi?id=247842 and https://bugzilla.mozilla.org/show_bug.cgi?id=228163).
It would be nice to have it in SeaMonkey 2.1

Comment 1

[Cédric Corazza]

Attachment: Patch for Help item menu without #ifdef

Comment 2

[[e-mail]]

Comment on attachment 405771 [details] [diff] [review]
Patch for Help item menu without #ifdef

It turns out that we can do better than this!

Because utilityOverlay.xul already includes platformCommunicatorOverlay.dtd as well as utilityOverlay.dtd, you can override any* utilityOverlay.xul string on a platform-specific basis by adding it to platformCommunicatorOverlay.dtd too.

i.e. in your locale, simply add this to win/platformCommunicatorOverlay.dtd:
<!-- Help viewer -->
<!ENTITY helpMenu.label                 "?">
<!ENTITY helpMenu.accesskey             "?">

(*The exceptions are the Edit menu entries in the bookmarks and history windows, because they use utilityOverlay.dtd directly.)

Comment 3

[Cédric Corazza]

(In reply to comment #2)
OK. But compare-locales script won't be happy with this. Though, the build process should start even if there are obsolete strings (strings not in en-US). But I'm not sure of this. That will also make the fr SeaMonkey tinderboxes forever orange.

Comment 4

[Cédric Corazza]

Attachment: Alternate patch

This is an alternate patch with less code and which won't break the build process and the tinderboxes wrt comment#2

Comment 5

[[e-mail]]

Comment on attachment 405776 [details] [diff] [review]
Alternate patch

>+<!-- LOCALIZATION NOTE some localizations of Windows use "?"
>+                       for the help button in the menubar but Gnome does not.   -->
Nit: This is a platform file, so I'm not sure that you need to mention Gnome.
Maybe something like
<!-- LOCALIZATION NOTE some localizations change the help label on Windows -->

>+<!ENTITY helpMenu.label                 "Help"> 
>+<!ENTITY helpMenu.accesskey             "H">
Nit: Please add these to the Help Viewer section as I indicated previously.

r=me with those fixed.

Comment 6

[Cédric Corazza]

Attachment: Final patch addressing Neil's comments [Checkin: Comment 11]

Comment 7

[Cédric Corazza]

(In reply to comment #7)
> It would be nice to have it in SeaMonkey 2.1
I wanted to say SeaMonkey 2.0.1

Comment 8

[Stefan [:stefanh]]

(In reply to comment #7)
> (In reply to comment #7)
> > It would be nice to have it in SeaMonkey 2.1
> I wanted to say SeaMonkey 2.0.1

You need to ask for "approval‑seamonkey2.0.1" (click the "details" link, then set the "?" flag on the attachment, and comment on the risk and why you want this on 2.0.1).

Comment 9

[Cédric Corazza]

Comment on attachment 405785 [details] [diff] [review]
Final patch addressing Neil's comments
[Checkin: Comment 11]

I tested the patch and it worked fine.
It would be nice to have so that we have consistent UI across Mozilla products.

Comment 10

[Robert Kaiser]

Comment on attachment 405785 [details] [diff] [review]
Final patch addressing Neil's comments
[Checkin: Comment 11]

Umm, Cedric, as a localizer you should know that we can't take L10n changes in a stable branch.

Comment 11

[Serge Gautherie (:sgautherie)]

Comment on attachment 405785 [details] [diff] [review]
Final patch addressing Neil's comments
[Checkin: Comment 11]


http://hg.mozilla.org/comm-central/rev/e0fb72d64bfe

Comment 12

[Cédric Corazza]

VERIFIED
Name=SeaMonkey
Version=2.1a1pre
BuildID=20091104002950