Closed Bug 239977 Opened 17 years ago Closed 16 years ago
Support ISO 639-2 code "tlh" for Klingon
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.6) Gecko/20040207 Firefox/0.8 Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.6) Gecko/20040207 Firefox/0.8 In February, the ISO 639 committee approved the language code "tlh" for Klingon (see URL given for code change notice). This should be added to Mozilla's knowledge of the ISO 639 code at some point. Klingon makes a nice demo of the capability of a browser to deal with tagged multilingual content; besides, there certainly are pages out there in Klingon. This can be done by adding the line tlh = Klingon to languageNames.properties in both the places it occurs in seamonkey (mozilla/xpfe/global/resources/locale/en-US/ languageNames.properties and mozilla/toolkit/locale/languageNames.properties ). This is nowhere near anything like approaching urgent. Reproducible: Always Steps to Reproduce:
Can you give some example URIs of pages in Klingon? I assume (and hope) that you are referring to Klingon in Latin script. See http://eikenes.alvestrand.no/pipermail/ietf-languages/2004-February/001735.html and other messages in the thread.
Sure. Wikipedia has some pages in Klingon which use "tlh" for the language code, for example: http://tlh.wikipedia.org/wiki/Hov_leng
Assignee: nobody → smontagu
Status: UNCONFIRMED → NEW
Component: Other → Internationalization
Ever confirmed: true
Product: Mozilla Localizations → Browser
QA Contact: amyy
Version: unspecified → Trunk
Why don't you add it to toolkit/locale/languageNames.properties as well? In addition, 'tlh' has to be added to intl/locale/src/languages.properties. And perhaps, 'tlh' has to be mapped to 'x-western' (for now) in 'langGroups.properties'
Apologies for that too hasty patch last night. This one should be better. By the way, in case anyone else tries to test putting Klingon at the top of their Accept-Language header and going to Google, there is a bug in Google's content negotiation (mentioned already in bug 208277 comment 4) which will make the results come out in Tagalog.
Attachment #161604 - Attachment is obsolete: true
Comment on attachment 161649 [details] [diff] [review] Patch including what was missing r=jshin thanks.
Attachment #161649 - Flags: review?(jshin) → review+
Comment on attachment 161649 [details] [diff] [review] Patch including what was missing Roc, can you sr? Will you give blanket rs for adding new language codes that get added to ISO-639 from time to time?
Attachment #161649 - Flags: superreview?(roc)
Comment on attachment 161649 [details] [diff] [review] Patch including what was missing Yes, I'll grant a blanket sr for adding these language codes.
Attachment #161649 - Flags: superreview?(roc) → superreview+
Fix checked in.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.