Closed Bug 378119 Opened 13 years ago Closed 12 years ago
unexpected space before auto-appended mnemonic on localized menu labels
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; ko-KR; rv:184.108.40.206) Gecko/20070310 Iceweasel/220.127.116.11 (Debian-18.104.22.168-2) Build Identifier: Mozilla/5.0 (X11; U; Linux x86_64; ko-KR; rv:22.214.171.124) Gecko/20070310 Iceweasel/126.96.36.199 (Debian-188.8.131.52-2) The Korean langpack never specifies menu mnemonics but there're mnemonics on every menu labels. I guess some code inserts mnemonics with parentheses (such like "File (_F)") to the menu labels when they don't have their own mnemonics. The problem is, it appends mnemonic text such like " (_F)" with a space between the labels and opening parentheses. But this is different with the convention other Korean applications (in MS-Windows/GNOME/KDE/...) are using. Those applications don't use space between the label and opening parentheses. Their labels just look like "File(_F)" without space. I don't know what convention is used for other languages. But at least in Korean desktop, Firefox looks a bit inconsistent because of this different menu mnemonic convention. Reproducible: Always Steps to Reproduce: 1. Start firefox with Korean locale. 2. Compare its menubar with other Korean app's menubar.
Does this happen with an official Firefox build from mozilla.com? This sounds like something that the distribution might change...
Whiteboard: CLOSEME - 06/15
Iceweasel has no such changes. I also use an "official" firefox in MS Windows and its menu labels have the same problem.
Is this something that's okay to fix?
I found this can be fixed in the Korean lang pack. By setting the default value of intl.menuitems.insertseparatorbeforeaccesskeys as false. (There seem to be some languages which require it?) Moving this bug to ko-KR localization product...
Component: Menus → ko-KR / Korean
Product: Firefox → Mozilla Localizations
(In reply to comment #4) > I found this can be fixed in the Korean lang pack. By setting the default > value of intl.menuitems.insertseparatorbeforeaccesskeys as false. (There seem > to be some languages which require it?) > > Moving this bug to ko-KR localization product... > This bug was old problem. Comparing with ja code, its value must be false. http://lxr.mozilla.org/l10n-mozilla1.8/source/ja/toolkit/chrome/global/intl.properties
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment on attachment 288303 [details] [diff] [review] intl.properties patch [ko] approving accesskey space removal for 184.108.40.206, please use the fixed220.127.116.11 and verified18.104.22.168 keywords to track landing and testing.
Attachment #288303 - Flags: approval22.214.171.124? → approval126.96.36.199+
fixed in 1.8.1 branch.
fixed in 1.9 trunk too.
You need to log in before you can comment on or make changes to this bug.