New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding search.query and search.search disposition #24668
Adding search.query and search.search disposition #24668
Conversation
FYI @gregorypappas |
Preview URLs
External URLs (3)URL:
(comment last updated: 2023-03-03 21:14:16) |
files/en-us/mozilla/add-ons/webextensions/api/search/search/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/api/search/search/index.md
Outdated
Show resolved
Hide resolved
This pull request has merge conflicts that must be resolved before it can be merged. |
@zombie would you have a moment to review? |
Co-authored-by: Gregory Pappas <email.greg.email@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The search.search
and search.query
methods are almost identical from the functionality perspective. The main difference is that search.query
is cross-browser, and that search.query
is Firefox-only and has been around for much longer AND that search.query
offers the ability to initiate a search query with a non-default search engine (via the engine
) property.
Could you document a comparison between the two to help readers with deciding which method to choose?
files/en-us/mozilla/add-ons/webextensions/api/search/search/index.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Rob Wu <rob@robwu.nl>
Description
Adds documentation for the
search.query
function and the addition of thedisposition
property tosearch.search
.Motivation
Addresses the documentation requirements of Bug 1804357 Implement chrome.search.
Additional details
BCD for this change was previously submitted in #18903