Closed Bug 2015461 Opened 9 months ago Closed 8 months ago

Allow reordering of hidden search engines

Categories

(Firefox :: Search, task, P1)

task

Tracking

()

VERIFIED FIXED
149 Branch
Tracking Status
firefox149 --- verified
firefox150 --- verified

People

(Reporter: standard8, Assigned: standard8)

References

Details

(Whiteboard: [sng])

Attachments

(1 external file)

With the new settings design, we'll be keeping hidden application provided engines available in the list of engines. The engines will still be shown as reorderable, and we think it'd be confusing for the user not to reorder them. Hence we should change the search service so that it stores the order for all engines (not just visible), and allow hidden engines to be moved.

This enables reordering for hidden search engines for the new settings table. We leave the existing
code in place for the pre-settings table, until we have turned that on completely, this is to avoid
breaking functionality in the meantime.

This also adds a more explicit test for moveEngine - some tests already test it a little, but not
extensively.

Pushed by mbanner@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/fd04c5066699 https://hg.mozilla.org/integration/autoland/rev/e297fbc620e2 Allow reordering of hidden search engines for the new settings UI. r=scunnane,tgiles
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 149 Branch
QA Whiteboard: [search] [qa-triage-done-c149/b148][qa-ver-needed-c149]
QA Whiteboard: [search] [qa-triage-done-c149/b148][qa-ver-needed-c149] → [search][qa-triage-done-c150/b149][qa-ver-needed-c150/b149]
Flags: qe-verify+

Verified as fixed with Firefox 150.0a1 (20260224095058) and Firefox 149.0b1 (20260225100912) on macOS 15 ARM, Ubuntu 22.04 and Windows 10.

Status: RESOLVED → VERIFIED
QA Whiteboard: [search][qa-triage-done-c150/b149][qa-ver-needed-c150/b149] → [search][qa-triage-done-c150/b149][qa-ver-done-c150/b149]
Flags: qe-verify+
QA Contact: dlucaci
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: