zh-TW locale update for Fx Aviary

RESOLVED FIXED

Status

RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: piaip, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 3 obsolete attachments)

(Reporter)

Description

13 years ago
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
(Reporter)

Comment 1

13 years ago
Created attachment 188299 [details]
Diff for aviary Fx, locale revised from trunk version
Attachment #188299 - Flags: approval-aviary1.0.5?
(Reporter)

Updated

13 years ago
Attachment #188299 - Attachment is obsolete: true
Attachment #188299 - Flags: approval-aviary1.0.5?
(Reporter)

Comment 2

13 years ago
Created attachment 188303 [details] [diff] [review]
cvsiff for Aviary Fx, toolkit/
Attachment #188303 - Flags: approval-l10n?
(Reporter)

Comment 3

13 years ago
Created attachment 188304 [details] [diff] [review]
cvsiff for Aviary Fx, browser/
Attachment #188304 - Flags: approval-l10n?

Updated

13 years ago
Attachment #188303 - Flags: approval-l10n? → approval-l10n-

Updated

13 years ago
Attachment #188304 - Flags: approval-l10n? → approval-l10n-

Comment 4

13 years ago
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.
(Reporter)

Comment 5

13 years ago
Created attachment 188313 [details] [diff] [review]
diff for toolkit

removed tool comments
Attachment #188303 - Attachment is obsolete: true
Attachment #188313 - Flags: approval-l10n?
(Reporter)

Comment 6

13 years ago
Created attachment 188314 [details] [diff] [review]
diff for browser

removed tool comments
Attachment #188304 - Attachment is obsolete: true
Attachment #188314 - Flags: approval-l10n?
(Reporter)

Comment 7

13 years ago
(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.

Updated

13 years ago
Attachment #188313 - Flags: approval-l10n? → approval-l10n+

Updated

13 years ago
Attachment #188314 - Flags: approval-l10n? → approval-l10n+
(Reporter)

Comment 8

13 years ago
Created attachment 189785 [details] [diff] [review]
patch for zh-TW Locale revised with CNS13188 standard.

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 9

13 years ago
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-
(Reporter)

Updated

13 years ago
Status: UNCONFIRMED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.