Closed
Bug 62015
Opened 24 years ago
Closed 24 years ago
Move pref "intl.collationKeyAsCodePoint" to chrome directory
Categories
(MailNews Core :: Internationalization, defect, P2)
MailNews Core
Internationalization
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9
People
(Reporter: ji, Assigned: nhottanscp)
Details
(Keywords: l12y)
Attachments
(1 file)
2.64 KB,
patch
|
Details | Diff | Splinter Review |
We need this pref to be able to sort correctly on linux. For 6.0, this is put in all.js in Japanese linux build.
Assignee | ||
Comment 1•24 years ago
|
||
Yes, it has to be moved to .property file and need to change the related .cpp file.
Status: NEW → ASSIGNED
Comment 2•24 years ago
|
||
Is it navigator.properties, then?
Assignee | ||
Comment 3•24 years ago
|
||
>Is it navigator.properties, then?
yes
Assignee | ||
Comment 5•24 years ago
|
||
intl -> l12y
Assignee | ||
Comment 6•24 years ago
|
||
"intl.collationKeyAsCodePoint" is a bool pref. There is no getLocalizedBoolPref() in nsIPref. Tao, do you think we should create getLocalizedBoolPref()? For this bug, I can change "intl.collationKeyAsCodePoint" to a char pref instead of bool.
Alecf owns the getLocalizedXX() api. IMO, your proposal will work. No need to add another api just for this particular bug.
Comment 8•24 years ago
|
||
Mass change of QA contact to ji for the bugs filed by her.
QA Contact: momoi → ji
Assignee | ||
Updated•24 years ago
|
Target Milestone: --- → mozilla0.8
Assignee | ||
Updated•24 years ago
|
Target Milestone: mozilla0.8 → mozilla0.9
Assignee | ||
Comment 9•24 years ago
|
||
Comment 10•24 years ago
|
||
r=tao if the default value of this pref is null in navigator.properties
Comment 11•24 years ago
|
||
How about a slightly better wording, for understandability: mKeyAsCodePoint -> mUseCodePointOrder collationKeyAsCodePoint -> useCodePointOrder Also, expand the comment in the property file to: valid collation options are: <empty string> or useCodePointOrder sr=erik
Assignee | ||
Comment 12•24 years ago
|
||
checked in
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 13•23 years ago
|
||
With 04/10 build, in the file of .../locale/en-US/navigator/navigator.properties I saw the pref of "intl.collationOption" Naoki, did you change it to a char pref or bool pref?
Assignee | ||
Comment 14•23 years ago
|
||
That's a char pref, the possible value are listed as in the file. 29 # valid collation options are: <empty string> or useCodePointOrder
Reporter | ||
Comment 15•23 years ago
|
||
Thanks. Verified as fixed. Added Ying and Ray to cc list. We need this pref set to useCodePointOrder for Japanese linux build.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•