Closed
Bug 266103
Opened 21 years ago
Closed 21 years ago
IT/sv/ru/pl not included in regionNames.properties
Categories
(Core :: Internationalization, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: Ervin.Yan, Assigned: Ervin.Yan)
Details
Attachments
(1 file, 4 obsolete files)
|
1.15 KB,
patch
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; zh-CN; rv:1.7) Gecko/20041004
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; zh-CN; rv:1.7) Gecko/20041004
dowload all spellchecking modules from:
http://dictionaries.mozdev.org/
Reproducible steps:
1. start Mozilla composer
2. click menu "Edit|Check Spelling"
3. click "Language:" combo box.
You will see it-IT/sv-SE/ru-RU/pl-PL not translated.
which means need include IT/sv/ru/pl in file:
xpfe/global/resources/locale/en-US/regionNames.properties
Reproducible: Always
Steps to Reproduce:
1.
2.
3.
|
||
Comment 2•21 years ago
|
||
Thank you for the patch. Please patch also toolkit/locale/regionNames.properties
and attach a contextual diff (made with diff -u).
Assignee: smontagu → Ervin.Yan
Attachment #163404 -
Attachment is obsolete: true
Comment on attachment 163512 [details] [diff] [review]
new patch
could you review the patch?
Attachment #163512 -
Flags: review?(smontagu)
|
|
||
Comment 5•21 years ago
|
||
Comment on attachment 163512 [details] [diff] [review]
new patch
r=smontagu
Attachment #163512 -
Flags: review?(smontagu) → review+
Attachment #163512 -
Flags: superreview?(dbaron)
Comment on attachment 163512 [details] [diff] [review]
new patch
The rest of the file uses tabs, so please use tabs. Otherwise, sr=dbaron,
although please check with bsmedberg before landing because of the aviary
branch merging.
Attachment #163512 -
Flags: superreview?(dbaron) → superreview+
|
||
Comment 8•21 years ago
|
||
In addition to the two places already listed, please also land this patch in
toolkit/locales/en-US/chrome/global/regionNames.properties
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attachment #163512 -
Attachment is obsolete: true
Attachment #166998 -
Attachment is obsolete: true
|
Assignee | |
Comment 10•21 years ago
|
||
bsmedberg: do you have any other concerns for landing this patch?
|
|
||
Comment 11•21 years ago
|
||
It looks fine to me.
|
|
Assignee | |
Comment 12•21 years ago
|
||
toolkit/locale/regionNames.properties is removed in latest trunk.
Attachment #167313 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 13•21 years ago
|
||
patch landed. thanks simford.dong for help to check in the patch.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•