Closed Bug 1499648 Opened 6 years ago Closed 5 years ago

Make search alias (@ token alias) formatting work with UrlbarView

Categories

(Firefox :: Address Bar, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 67
Tracking Status
firefox67 --- fixed

People

(Reporter: dao, Assigned: adw)

References

Details

(Whiteboard: [fxsearch])

Attachments

(1 file)

This is currently disabled because this.urlbarInput.popup doesn't exist (there's UrlbarInput::panel and UrlbarInput::view), but more fundamentally I think we're missing APIs such as popup.selectedIndex and popup.oneOffSearchButtons.

https://searchfox.org/mozilla-central/rev/3d989d097fa35afe19e814c6d5bc2f2bf10867e1/browser/components/urlbar/UrlbarValueFormatter.jsm#275,286,289
Summary: Make search alias formatting work with UrlbarView → Make search alias (@ token alias) formatting work with UrlbarView
Assignee: nobody → adw
Status: NEW → ASSIGNED
See Also: → 1523774
Blocks: 1524125

assertHighlighted() in browser_urlbarTokenAlias.js currently skips its check in quantumbar, so that needs to be fixed along with this bug.

Depends on: 1525269, 1502455

This is based on bugs 1528430, 1527947, and 1527934.

Pushed by dwillcoxon@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0dbdcaebed42
Make search alias (@ token alias) formatting work with UrlbarView. r=dao
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: