Closed
Bug 296136
Opened 20 years ago
Closed 20 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•20 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•20 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•20 years ago
|
Attachment #190559 -
Flags: superreview?(mozilla)
Attachment #190559 -
Flags: review?(mozilla)
Comment 3•20 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•20 years ago
|
||
Did this ever get checked in?
| Assignee | ||
Comment 5•20 years ago
|
||
No, I can see it in CVS still unpatched.
Updated•20 years ago
|
Assignee: mozilla → mozilla
| Assignee | ||
Updated•20 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•