Closed Bug 1986890 Opened 1 year ago Closed 2 months ago

Support "add engine" badge on the unified search button

Categories

(Firefox :: Address Bar, task, P3)

task
Points:
5

Tracking

()

VERIFIED FIXED
155 Branch
Tracking Status
firefox155 --- fixed
firefox157 --- verified
firefox158 --- verified

People

(Reporter: dao, Assigned: dao)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [sng][scotchbonnet-followon])

Attachments

(1 file)

For the search bar, we'll likely want to display the "add engine" badge on the button when a web page offers an engine.

Summary: Support "add engine" bade on the unified search button → Support "add engine" badge on the unified search button
Duplicate of this bug: 2007426
Priority: P2 → P3
Blocks: 2002830
No longer blocks: 1974218
Priority: P3 → P1
Priority: P1 → P3
Whiteboard: [sng] → [sng][scotchbonnet-followon]

The old search bar (browser.search.widget.new=false) badges its search button
when the current page offers an opensearch engine. Do the same for the new
search bar's unified search button, driven from the same OpenSearchManager path
that already updates the old search bar.

The badge is scoped to the search bar; the address bar's unified search button
stays unbadged, matching the old behavior.

Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Pushed by dgottwald@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/0e283f94bbe9 https://hg.mozilla.org/integration/autoland/rev/92d08efec462 Show the "add engine" badge on the unified search button of the new search bar. r=daleharvey

Backed out for causing build bustages
Backout Link
Push with failures
Failure Log
Failure line Error processing test manifest file /builds/worker/checkouts/gecko/browser/components/urlbar/tests/browser/searchbar/browser.toml: Traceback (most recent call last):

Flags: needinfo?(dao+bmo)
Pushed by dgottwald@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/16e8979074c6 https://hg.mozilla.org/integration/autoland/rev/5278b4c30fe4 Show the "add engine" badge on the unified search button of the new search bar. r=daleharvey
Pushed by asilaghi@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/16a3d8ab97fe https://hg.mozilla.org/integration/autoland/rev/802369548217 Revert "Bug 1986890 - Show the "add engine" badge on the unified search button of the new search bar. r=daleharvey" for causing doc generate failures

Backed out for causing doc generate failures
Backout Link
Push with failures
Failure Log
Failure line sphinx.errors.ExtensionError: Handler <function analyze at 0x7f242fbdaca0> for event 'builder-inited' threw an exception (exception: Your code contains multiple documented objects at each of these paths:

Attachment #9608514 - Attachment description: Bug 1986890 - Show the "add engine" badge on the unified search button of the new search bar. r?daleharvey → Bug 1986890 - Show the "add engine" badge on the unified search button of the new search bar. r=daleharvey
Flags: needinfo?(dao+bmo)
Pushed by dgottwald@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/121994d3fa97 https://hg.mozilla.org/integration/autoland/rev/530da3d51cd5 Show the "add engine" badge on the unified search button of the new search bar. r=daleharvey
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 155 Branch
QA Whiteboard: [search][qa-triage-done-c155/b154][qa-ver-needed-c155/b154]

Reproducible on a 2026-07-27 Firefox Nightly build on Windows 10.

Verified as fixed on Firefox Nightly 158.0a1 and Firefox 157.0b5 on Windows 10, Ubuntu 22, macOS 13.

Status: RESOLVED → VERIFIED
QA Whiteboard: [search][qa-triage-done-c155/b154][qa-ver-needed-c155/b154] → [search][qa-triage-done-c155/b154][qa-ver-done-c158/b157]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: