Closed Bug 1469688 Opened Last year Closed Last year

Allow reordering the precedence of installed language packs

Categories

(Firefox :: Preferences, enhancement, P2)

enhancement

Tracking

()

VERIFIED FIXED
Firefox 63
Tracking Status
firefox63 --- verified
firefox64 --- verified
firefox65 --- verified
firefox66 --- verified

People

(Reporter: mstriemer, Assigned: mstriemer)

References

(Blocks 1 open bug)

Details

Attachments

(5 files)

When a user has multiple language packs installed they might want to reorder which languages they would like to see. This UI can be quite similar to the webpage languages dialog.

There is a mock for a nicer UI for this, so try to re-use the webpage dialog in a way that they can both be updated later.
Blocks: 1469690
No longer blocks: 1469690
Assignee: nobody → mstriemer
Attached image reorder-locales.mov.gif
Comment on attachment 8999312 [details]
Bug 1469688 - Part 2: Allow reordering of browser languages in prefs r?Gijs,zbraniecki,flod

Francesco Lodolo [:flod] has approved the revision.
Attachment #8999312 - Flags: review+
Comment on attachment 8999311 [details]
Bug 1469688 - Part 1: Generalize language switching code in prefs r?jaws,zbraniecki

Zibi Braniecki [:gandalf][:zibi] has approved the revision.
Attachment #8999311 - Flags: review+
Comment on attachment 8999311 [details]
Bug 1469688 - Part 1: Generalize language switching code in prefs r?jaws,zbraniecki

(away 8/20-8/27) Jared Wein [:jaws] (please needinfo? me) has approved the revision.
Attachment #8999311 - Flags: review+
Comment on attachment 8999312 [details]
Bug 1469688 - Part 2: Allow reordering of browser languages in prefs r?Gijs,zbraniecki,flod

Zibi Braniecki [:gandalf][:zibi] has approved the revision.
Attachment #8999312 - Flags: review+
Attachment #8999312 - Flags: review?(gijskruitbosch+bugs)
I've updated based on your comments, Gijs, and managed to remove some of the code. Thanks for the review!
Comment on attachment 8999312 [details]
Bug 1469688 - Part 2: Allow reordering of browser languages in prefs r?Gijs,zbraniecki,flod

:Gijs (he/him) has approved the revision.
Attachment #8999312 - Flags: review+
Attachment #8999312 - Flags: review?(gijskruitbosch+bugs)
Attachment #8999312 - Attachment description: Bug 1469688 - Part 2: Allow reordering of browser languages in prefs r?jaws,zbraniecki → Bug 1469688 - Part 2: Allow reordering of browser languages in prefs r?Gijs,zbraniecki,flod
Comment on attachment 9004696 [details]
Bug 1469688 - Part 3: Maintain the order of browser languages dialog r?Gijs

:Gijs (he/him) has approved the revision.
Attachment #9004696 - Flags: review+
Comment on attachment 9004697 [details]
Bug 1469688 - Part 4: Test the browser languages dialog r?Gijs

:Gijs (he/him) has approved the revision.
Attachment #9004697 - Flags: review+
Pushed by mstriemer@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/505226ad06ab
Part 1: Generalize language switching code in prefs r=jaws,zbraniecki
https://hg.mozilla.org/integration/mozilla-inbound/rev/6edbb5f98d6e
Part 2: Allow reordering of browser languages in prefs r=Gijs,zbraniecki,flod
https://hg.mozilla.org/integration/mozilla-inbound/rev/bcec32bfccff
Part 3: Maintain the order of browser languages dialog r=Gijs
https://hg.mozilla.org/integration/mozilla-inbound/rev/55662a7bc29c
Part 4: Test the browser languages dialog r=Gijs
Blocks: 1488099
Could you please provide some steps to follow in order to confirm this?

I cannot find the drop-down in the language section, the only button available is to choose the web pages display language.
I have also tried to see if it activates by adding different language packs to the browser, but without success.
Flags: needinfo?(mstriemer)
@cristian
We have a PI request tracking the whole feature (Cristian Craciun is coordinating), not sure if we need to verify each bug?

The feature is behind the intl.multilingual.enabled flag.
Flags: needinfo?(mstriemer)
This Is Confirmed as fixed in Firefox 64.0, Nightly 66 and Beta 65.0b6.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Depends on: 1539137
You need to log in before you can comment on or make changes to this bug.