1972082 - Convert Website languages to config-based prefs box group

Status: RESOLVED FIXED

(Firefox :: Settings UI, enhancement, P3)

Description

[Mark Striemer [:mstriemer]]

Create a box-group that replaces the existing Webpage Language Settings dialog.

This is an enhancement on bug 1972081 which moves the language management into the top-level page and out of its existing dialog.

https://www.figma.com/design/2b02v18AppIpsTceWac2T3/Settings-Desktop?node-id=6755-16045&t=QyyPgM3bfhGhjKIE-4

Comment 1

[Tim Giles [:tgiles]]

Attachment: Bug 1972082 - Convert Website languages to config-based box group. r=#recomp-reviewers,#translations-reviewers

init commit

Fix the website language setting group not rendering

update l10nId for language picker wrapper

Comment 2

[Tim Giles [:tgiles]]

Attachment: WIP: Bug 1972082 setting controls are not reorderable in box group

I'm not sure how to make the config work with the designed layout.
It seems like adding controls breaks the reorderable box since
the markup goes: moz-box-group > setting-control > moz-box-item.

I'm sure I'm just not writing the config in the correct way,
especially as the reorderable box items are going to be
dynamically generated anyway.

Comment 3

[Tim Giles [:tgiles]]

Attachment: WIP: Bug 1972082 - Why is the option markup not updating

Comment 4

[Tim Giles [:tgiles]]

Attachment: WIP: Bug 1972082 - Try to fix test_moz_box_group reordering

squash: start fixing test_moz_box_group.html

I don't think I've implemented the wrapper element correctly because the
moz-reorderable-list logic is preventing the 'reorder' event from
firing.

Comment 5

[Hanna Jones [:hjones]]

Attachment: Screenshot 2026-01-21 at 10.44.59 AM.png

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/c8e458b97278
https://hg.mozilla.org/integration/autoland/rev/397d4239c8df
Convert Website languages to config-based box group. r=fluent-reviewers,akulyk,bolsson

Comment 7

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/397d4239c8df