Closed Bug 294286 Opened 19 years ago Closed 9 days ago

cmdkey characters are shown as MacRoman characters in menu

Categories

(Core Graveyard :: Widget: Mac, defect)

1.8 Branch
PowerPC
macOS
defect
Not set
major

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: stefanh, Unassigned)

References

Details

(Keywords: intl, l12y, platform-parity)

Attachments

(1 file)

Tested on recent FF trunk with Mac OS 10.3.9:

It appears that the commandkey characters are not interpreted as Unicode
characters in Firefox on Mac OS X.

Using the Swedish characters "Ä", ie "Ä"(196) and "Ö", ie "Ö"(214) as
commandkeys will produce the equivalent MacRoman characters in the menu: ƒ
(196) and ÷ (214). The labels etc works fine.

This is an issue for any localizer that, for instance, doesn't have easy access
to [ and ] (on a Swedish keyboard you'll have to press Option+8 in order to
produce a "[")

biesi had an idea of the problem:

<biesi>	ok
	I'd say the bug is at
http://lxr.mozilla.org/seamonkey/source/widget/src/mac/nsMenuX.cpp#272

<biesi>	the ToCString there truncates the unicode value to a latin1 char
	I assume SetItemCmd interprets it in the system charset
Here's a test-patch to show the issue. Note that the labels works fine.
Ah, forgot to say that the shortcuts works, the issue is only with the displayed
characters.
Blocks: 294316
Severity: normal → major
Keywords: l12y
Don't put LXR links into bug descriptions. When the file changes, it doesn't
make sense any more. Better to just copy/paste the line with some context and a
file name.
Blocks: 306585
The cocoa switch seems to have fixed this
Assignee: joshmoz → nobody
This doesn't really block bug 306585. I originally made it block because I wanted to use those characters instead of being forced to use the non-working en-US ones (even if they would have worked, they're not really an option).
No longer blocks: 306585
This 1.8-only - the cocoa switch fixed this.
Version: Trunk → 1.8 Branch
Product: Core → Core Graveyard
Status: NEW → RESOLVED
Closed: 9 days ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: