Closed Bug 299712 Opened 19 years ago Closed 19 years ago

zh-TW locale update for Fx Aviary

Categories

(Mozilla Localizations :: Other, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: piaip, Unassigned)

Details

Attachments

(3 files, 3 obsolete files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; zh-TW; rv:1.7.8) Gecko/20050511 Firefox/1.0.4
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; zh-TW; rv:1.7.8) Gecko/20050511 Firefox/1.0.4

It has been a long time since Fx1.0 release and we had some improvements to
the zh-TW locale in trunk. Here is the diff for Aviary.

Reproducible: Always
Attachment #188299 - Flags: approval-aviary1.0.5?
Attachment #188299 - Attachment is obsolete: true
Attachment #188299 - Flags: approval-aviary1.0.5?
Attached patch cvsiff for Aviary Fx, toolkit/ (obsolete) — Splinter Review
Attachment #188303 - Flags: approval-l10n?
Attached patch cvsiff for Aviary Fx, browser/ (obsolete) — Splinter Review
Attachment #188304 - Flags: approval-l10n?
Attachment #188303 - Flags: approval-l10n? → approval-l10n-
Attachment #188304 - Flags: approval-l10n? → approval-l10n-
We're only taking security related fixes for Aviary. And we're not taking patches
that contain just tool comments in half of the files. Please fix your tool to
stop changing unchanged files.

This really looks like 1.1 stuff to me, but of course, I can't read what you 
actually did.
Attached patch diff for toolkitSplinter Review
removed tool comments
Attachment #188303 - Attachment is obsolete: true
Attachment #188313 - Flags: approval-l10n?
Attached patch diff for browserSplinter Review
removed tool comments
Attachment #188304 - Attachment is obsolete: true
Attachment #188314 - Flags: approval-l10n?
(In reply to comment #4)
> And we're not taking patches contain just tool comments in half of the files.
> Please fix your tool to stop changing unchanged files.
 I apologize for that. Forgot to check the content...
 New versions are uploaded.

> This really looks like 1.1 stuff to me, but of course, 
> I can't read what you actually did.
 Let me explain what are these.
 Most of them are to solve a symbol \uFF1A. It is supposed to be look like ':' 
 in DBCS width. However some people using Lucida Grande and other fonts as their
 user interface on Windows will see a strange symbol for that.
 As a result, we decided to change it back to ':'.

 Several of entries are wrong (or not good) in translation. All of are updated
 from a centralized locale database so they are not 1.1, all made for 1.0.

 If we are going to release 1.0.5 maybe these can make 1.0.5 better. However
 I know these are not really security fixes so it's OK if you think these 
 should be rejected.

 Thanks for your review.
Attachment #188313 - Flags: approval-l10n? → approval-l10n+
Attachment #188314 - Flags: approval-l10n? → approval-l10n+
The zh-TW locale was revised by some official team for a standard in TW
named "CNS13188" for language/region/locale names in translation.
Also some typos (for example, browser/pref/pref-advanced.dtd)
and improvements were included according to same standard recommendation.
Attachment #189785 - Flags: approval-l10n?
Comment on attachment 189785 [details] [diff] [review]
patch for zh-TW Locale revised with CNS13188 standard.

1.5 is at the door, not going to take this anymore.
Attachment #189785 - Flags: approval-l10n? → approval-l10n-
Status: UNCONFIRMED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: