Closed Bug 152790 Opened 22 years ago Closed 22 years ago

tis620-2 is not defined as th language group

Categories

(Core :: Internationalization, defect)

Sun
Solaris
defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: masaki.katakai, Assigned: masaki.katakai)

References

(Blocks 1 open bug)

Details

(Keywords: intl)

Attachments

(2 files)

tis620-2 is not defined as th (Thai) language group in charsetData.properties.
So when we bring up font preference dialog, there is no font available for
Thai language.
Attached patch patch (deleted) — Splinter Review
Blocks: thai
Status: NEW → ASSIGNED
Summary: tis620-2 is not defined as th language group → tis620-2 is not defined as th language group
Yokoyama-san,

could you please r= for the patch?

Thank you.
just reminder,

tis-620 is for character encoding.
tis-620-x (tis-620-0, tis-620-1, tis-620-2) is for font set.
Keywords: intl
QA Contact: ruixu → ylong
Can anyone r= please?
Comment on attachment 88282 [details] [diff] [review]
patch

r=ftang make sense.
Attachment #88282 - Flags: review+
Comment on attachment 88282 [details] [diff] [review]
patch

sr=sfraser
Attachment #88282 - Flags: superreview+
Thanks for review and super-review.

Sorry, attachment 88282 [details] [diff] [review] contains other changes (hkscs-1) for testing purpose
by mistake, only tis620-2 line is valid.

Checked into Trunk.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Keywords: mozilla1.0.1
Attached patch patch for branch (corrected) (deleted) — Splinter Review
Katakai-san:
Do I need some thai font available in my linux system in order to verify this?
if so, could you tell me how/where can I get some thai fonts?  Thanks!
Yes... but this problem happened with ctl enabled binary.
So it's too difficult to verify.

I've assgined QA to Arthit.

Arthit, I'll send you the binary enabled CTL.
Please verify on the binary.
QA Contact: ylong → arthit.suriyawongkul
tested the binary from Masaki, the issue has been fixed.
verified.
Status: RESOLVED → VERIFIED
Comment on attachment 88282 [details] [diff] [review]
patch

a=chofmann for 1.0.1.  add the fixed1.0.1 keyword after checking into the
branch
Attachment #88282 - Flags: approval+
checked into branch.

Thank you very much for approval.
arthit: can you verify this bug fix in 1.01 branch?  When verified, pls replace
fixed1.0.1 keyword with verified1.0.1.  Thanks.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: