Closed
Bug 296136
Opened 19 years ago
Closed 19 years ago
Add necessary lines to os2charset.properties
Categories
(Core :: Internationalization, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: mozilla, Assigned: mozilla)
Details
Attachments
(1 file)
489 bytes,
patch
|
mkaply
:
review+
mkaply
:
superreview+
mkaply
:
approval1.8b4+
|
Details | Diff | Splinter Review |
There was a thread in the n.p.m.os2 newsgroup in April titled "Mozilla and codepage 921" about non-displaying characters on Baltic (iso-8859-13) webpages if the system was set up with codepage 921. The problem was obviously solved by adding the line os2.921=ISO-8859-13 to intl/uconv/src/os2charset.properties. I guess we need to get this in the tree. Are any other lines missing? E.g. why is there no line for ISO-8859-2? Is this OS/2 file on to override good defaults for other platforms?
Comment 1•19 years ago
|
||
This file is basically serving one purpose - map system charsets to Mozilla charsets. Each platform has this file. So what should be in this file is charsets that people use as their system code page and their Mozilla equivalents.
Assignee | ||
Comment 2•19 years ago
|
||
OK, I found out that the assignments in this file are already good for e.g. Polish codepage (cp852) and nobody yet complained about anything else than cp921, so it's probably enough to add this for now.
Assignee | ||
Updated•19 years ago
|
Attachment #190559 -
Flags: superreview?(mozilla)
Attachment #190559 -
Flags: review?(mozilla)
Comment 3•19 years ago
|
||
Comment on attachment 190559 [details] [diff] [review] Add codepage 921 r/sr/a=mkaply (OS/2 only)
Attachment #190559 -
Flags: superreview?(mozilla)
Attachment #190559 -
Flags: superreview+
Attachment #190559 -
Flags: review?(mozilla)
Attachment #190559 -
Flags: review+
Attachment #190559 -
Flags: approval1.8b4+
Comment 4•19 years ago
|
||
Did this ever get checked in?
Assignee | ||
Comment 5•19 years ago
|
||
No, I can see it in CVS still unpatched.
Updated•19 years ago
|
Assignee: mozilla → mozilla
Assignee | ||
Updated•19 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•