Closed
Bug 205649
Opened 21 years ago
Closed 21 years ago
convert U+00E5 in ISO-8859-1 in xul files to UTF-8
Categories
(SeaMonkey :: UI Design, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jshin1987, Assigned: jshin1987)
Details
Attachments
(1 file)
7.07 KB,
patch
|
hwaara
:
review+
|
Details | Diff | Splinter Review |
While working on bug 91190, I found that some xul files (UTF-8) have Latin characters (it's U+00E5 in comment). Although U+00E5 is in comment, I thought it's a good idea to convert them all to UTF-8.
Assignee | ||
Comment 1•21 years ago
|
||
The patch is to convert 0xE5(U+00E5 in ISO-8859-1) to its proper UTF-8 representation in several files. I'm also inserting U+00E5 in UTF-8 where it's missing. That is, I'm replacing 'Hkan' with 'Håkan'. The patch is in a mixed encoding because the original is in ISO-8859-1 while the replacement is in UTF-8. (this comment is in UTF-8)
Assignee | ||
Comment 2•21 years ago
|
||
Comment on attachment 123270 [details] [diff] [review] a patch This is just fixing nits. I'm asking Håkan (whose name is involved :-) for r and David for sr. I believe sr is not necessary for this type of patch, but just in case, a quick sr stamp woud be nice. Thanks.
Attachment #123270 -
Flags: superreview?(hyatt)
Attachment #123270 -
Flags: review?(hwaara)
Comment 3•21 years ago
|
||
Comment on attachment 123270 [details] [diff] [review] a patch As long as it's still a 'å' in the other encoding, sure.
Attachment #123270 -
Flags: review?(hwaara) → review+
.
Assignee: hyatt → jshin
Component: XP Toolkit/Widgets: XUL → XP Apps: GUI Features
QA Contact: shrir → paw
Summary: convert U+00E5 in ISO-8859-1 in xul files to UTF-8 → convert U+00E5 in ISO-8859-1 in xul files to UTF-8
Assignee | ||
Comment 5•21 years ago
|
||
Comment on attachment 123270 [details] [diff] [review] a patch This is just fixing comments so that I went ahead and checked it in.
Attachment #123270 -
Flags: superreview?(hyatt)
Assignee | ||
Comment 6•21 years ago
|
||
marking as fixed
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: Core → Mozilla Application Suite
You need to log in
before you can comment on or make changes to this bug.
Description
•