Closed Bug 1968110 Opened 1 year ago Closed 7 months ago

Convert settings related to Search page to config-based settings

Categories

(Firefox :: Settings UI, task)

task

Tracking

()

RESOLVED FIXED
151 Branch
Tracking Status
firefox151 --- fixed

People

(Reporter: mstriemer, Assigned: scunnane)

References

(Depends on 3 open bugs, Blocks 2 open bugs)

Details

(Whiteboard: [recomp][sng][srd-followups])

Attachments

(1 external file)

The Search page will need to be converted to the config-based settings structure. If settings share a similar pattern to the current design then it may make sense to convert them to the SRD 2025's new design. If they differ significantly, it may be necessary to create a custom component for the old version, and a new version; ideally this can still share most of the business logic to avoid regressions later.

This page will have the following sections which may be good candidates for individual bugs to work on:

  • Default search engine
  • Search Suggestions
  • Address Bar - Firefox Suggest
  • Search Shortcuts
Depends on: 1972068
Depends on: 1972069
Depends on: 1972071
Depends on: 1972072
Depends on: 1973527

Once all the blocking conversions are complete this bug will cover the work involved in reorganizing the relevant sections to create the finalized Search page as specified in Figma here: https://www.figma.com/design/2b02v18AppIpsTceWac2T3/Settings-Desktop?node-id=6396-14169&m=dev

As part of this work we should also make sure all the strings and illustrations match the design.

Duplicate of this bug: 1996606
Depends on: 2006881
Depends on: 2007397
Depends on: 2012186
Blocks: 2013042
No longer depends on: 2006881
Depends on: 2013054
Depends on: 2013055
Depends on: 2013056
Depends on: 2013831
Depends on: 2015461
Blocks: 1973527
No longer depends on: 1973527
Depends on: 2016088
Depends on: 2016111
Depends on: 2017287

To complete the work on this ticket we need to start generating the new Search settings pane via config. There's an example of us doing this work for the Account and Sync pane here. Basically the steps are as follows:

  • create a new browser/components/preferences/config/search.mjs file for all of the new/config-based search code
  • migrate any code needed for config-based search settings to that file from search.js and main.js (e.g. config like this and all of these addSetting calls), then import that file back into search.js (this will allow the old code to keep working as expected)
    • this will allow us to easily remove the old code down the line by just deleting the search.js file
  • add a new entry for the search pane to CONFIG_PANES here
    • more info on the various config options can be found here
    • it looks like the order of the settings may have changed in the mockups - implementing this should be as simple as changing the order of items in the groupIds array
  • do a final string and illustration check to make sure all of the copy is up to date

If you run into any unexpected issues just reach out in #the settings-redesign channel

Whiteboard: [recomp] → [recomp][sng][srd-followups]
Assignee: nobody → scunnane
Attachment #9566417 - Attachment description: WIP: Bug 1968110 - Generate new search settings pane via config. r?Standard8! → WIP: Bug 1968110 - Generate new search settings pane via config. r?hjones!,Standard8
Attachment #9566417 - Attachment description: WIP: Bug 1968110 - Generate new search settings pane via config. r?hjones!,Standard8 → Bug 1968110 - Generate new search settings pane via config. r?hjones!,Standard8
See Also: → 2031341
Pushed by scunnane@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/cc0c7ef0882a https://hg.mozilla.org/integration/autoland/rev/38ad80672070 Generate new search settings pane via config. r=Standard8,hjones,fluent-reviewers,flod
Status: NEW → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → 151 Branch
QA Whiteboard: [qa-triage-done-c152/b151]
Blocks: 1546652
Blocks: 1138112
Blocks: 1251982
Blocks: 1345185
Blocks: 1446359
Blocks: 1433263
Blocks: 1439521
Blocks: 1522816
Blocks: 1652458
Blocks: 1722632
Blocks: 1959913
Blocks: 1971099
No longer blocks: 1959913
Blocks: 1149093
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: