The default bug view has changed. See this FAQ.

Language button has a label referencing location of intl.properties

VERIFIED FIXED in Firefox 9

Status

Fennec Graveyard
General
VERIFIED FIXED
6 years ago
5 years ago

People

(Reporter: aaronmt, Assigned: wesj)

Tracking

({regression})

Trunk
Firefox 9
ARM
Android
regression

Details

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
Created attachment 562293 [details]
Nightly (09/25)

Mozilla/5.0 (Android; Linux armv7l; rv:9.0a1) Gecko/20110925 Firefox/9.0a1 Fennec/9.0a1

In preferences, the 'Language' button has a label with a value: chrome://global/locale/intl.properties.

See screenshot.
(Assignee)

Updated

6 years ago
Assignee: nobody → wjohnston
Duplicate of this bug: 689092
(Assignee)

Comment 2

6 years ago
Created attachment 562464 [details] [diff] [review]
Patch

My original concern here was when users download and install a locale and then return to the Prefs pane. We don't have a nice language string to show, so instead we fall back to showing the locale code. Apparently when we're using auto-detection that can result in this strangeness?

This makes that fallback only happen if the pref has a user set value.

Alternatively, instead of listening for pref change notifications, I could fire off an observer notification from the locale picker when "Continue" (or Cancel, etc) was pressed that had the current selected locale name attached.
Attachment #562464 - Flags: review?(mark.finkle)
Comment on attachment 562464 [details] [diff] [review]
Patch

>-    let selectedLocale = Services.prefs.getCharPref("general.useragent.locale");
>+    let selectedLocale = chrome.getSelectedLocale("browser");
>+    // the chrome locale may not have updated yet if the user is installing a new
>+    // locale. if the pref has a user set value, use it instead

Add a blank line before the comment
Attachment #562464 - Flags: review?(mark.finkle) → review+
tracking-fennec: --- → ?
status-firefox9: --- → affected
https://hg.mozilla.org/mozilla-central/rev/65ae0b5132e5
Status: NEW → RESOLVED
Last Resolved: 6 years ago
status-firefox9: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 9
Verified fixed on:
Mozilla/5.0 (Android;Linux armv7l;rv:9.0a1)Gecko/20110927
Firefox/9.0a1 Fennec/9.0a1
Device: Acer ICONIA A500
OS: Android 3.1
Status: RESOLVED → VERIFIED
tracking-fennec: ? → -
Keywords: regressionwindow-wanted
You need to log in before you can comment on or make changes to this bug.