Closed Bug 521647 Opened 10 years ago Closed 10 years ago

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

Categories

(SeaMonkey :: OS Integration, defect)

x86
Windows XP
defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED
seamonkey2.1a1

People

(Reporter: cedric.corazza, Assigned: cedric.corazza)

Details

(Keywords: l12y)

Attachments

(1 file, 3 obsolete files)

Attached patch Patch for Help item menu (obsolete) — Splinter Review
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
Attachment #405760 - Flags: review?
Attachment #405760 - Attachment is obsolete: true
Attachment #405771 - Flags: review?
Attachment #405760 - Flags: review?
Attachment #405771 - Flags: review? → review?(neil)
Attachment #405771 - Flags: review?(neil) → review-
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.)
(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.
Attached patch Alternate patch (obsolete) — Splinter Review
This is an alternate patch with less code and which won't break the build process and the tinderboxes wrt comment#2
Attachment #405776 - Flags: review?(neil)
Attachment #405776 - Flags: review?(neil) → review+
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.
Attachment #405771 - Attachment is obsolete: true
Whiteboard: [checkin-needed sm2.1]
(In reply to comment #7)
> It would be nice to have it in SeaMonkey 2.1
I wanted to say SeaMonkey 2.0.1
Whiteboard: [checkin-needed sm2.1] → [checkin-needed sm2.0.1]
(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).
Whiteboard: [checkin-needed sm2.0.1]
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.
Attachment #405785 - Flags: approval-seamonkey2.0.1?
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.
Attachment #405785 - Flags: approval-seamonkey2.0.1? → approval-seamonkey2.0.1-
Keywords: checkin-needed
Comment on attachment 405785 [details] [diff] [review]
Final patch addressing Neil's comments
[Checkin: Comment 11]


http://hg.mozilla.org/comm-central/rev/e0fb72d64bfe
Attachment #405785 - Attachment description: Final patch addressing Neil's comments → Final patch addressing Neil's comments [Checkin: Comment 11]
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a1
Attachment #405776 - Attachment is obsolete: true
VERIFIED
Name=SeaMonkey
Version=2.1a1pre
BuildID=20091104002950
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.