Closed Bug 268792 Opened 20 years ago Closed 20 years ago

en-GB locale - Preferences - Languages in order of preference list is empty

Categories

(Firefox :: Settings UI, defect)

1.0 Branch
defect
Not set
minor

Tracking

()

RESOLVED FIXED

People

(Reporter: cnst+bmo, Assigned: cnst+bmo)

Details

(Keywords: intl)

Attachments

(2 files)

Preferences - General - Languages:
The list of the languages in order of preference appears to be empty. I think
that the problem is linked with the capitalisation of the
"intl.accept_languages" preference.

Right now it is defined in mozilla/browser/locales/en-GB/firefox-l10n.js in the
following way:
pref("intl.accept_languages", "en-GB, en"); 

If in about:config I twiddle the string to "en-gb, en", then the list of
preferred languages appears as intended.
Attachment #165381 - Flags: approval-aviary?
Comment on attachment 165381 [details] [diff] [review]
patch for en-GB/firefox-l10n.js

that pref is set in 
http://lxr.mozilla.org/aviarybranch/source/browser/app/profile/firefox.js#240
and the en-GB locale version of intl.properties is ok. the pref in
firefox-l10n.js is just wrong and should be removed completely
Attachment #165381 - Flags: review-
Attachment #165381 - Flags: approval-aviary?
Attachment #165381 - Flags: approval-aviary-
Comment on attachment 165415 [details] [diff] [review]
removed pref from firefox-l10n.js

requesting approval-aviary
Attachment #165415 - Flags: approval-aviary?
Attachment #165415 - Flags: approval-aviary? → approval-aviary+
fix checked in.
Please verify.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
I don't have the privileges to mark this VERIFIED, but it WFM on both Win32
installer & zip (clean profiles each time), with the below UA (after fix
check-in + daily build).

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-GB; rv:1.7.5) Gecko/20041110 Firefox/1.0
sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs,
filter on "beltznerLovesGoats" to get rid of this mass change
QA Contact: mconnor → preferences
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: