Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Brave should allow search engines to be added via OpenSearch #10868

Closed
bsclifton opened this issue Sep 8, 2017 · 8 comments
Closed

Brave should allow search engines to be added via OpenSearch #10868

bsclifton opened this issue Sep 8, 2017 · 8 comments
Labels
feature/search fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. needs-mockup A feature which needs design mockup to be implemented. settings wontfix

Comments

@bsclifton
Copy link
Member

bsclifton commented Sep 8, 2017

Did you search for similar issues before submitting this one?

Yes

Describe the issue you encountered:

Per #3053 (comment) by @da2x:

All search engines, and then I mean every single one of them, have OpenSearch auto-discovery on their home pages and search result pages. Safari and Edge automatically detects OpenSearch-enabled search engines and lists them as suggested search engines in their Preference panes.

This means that every user will find their favorite search engine in the list of possible search engines without having to configure it; they just need to perform one search with their favorite engine and it will then become available. I wrote about this on my blog for anyone interested in the specifics.

I think this is an elegant solution that empowers even inexperienced users to change their search engine to whichever they prefer without technical knowledge. It also exposes many more possible search engines than a browser could reasonably build-in by default. The approach sounds like a good fit for Brave.

Example steps outlined (showing what to look for):

Platform (Win7, 8, 10? macOS? Linux distro?):

All

Brave Version (revision SHA):

All

Steps to reproduce:

1. Visit https://translate.google.com
2. Open Preferences > Search
3. Notice that `translate.google.com` was not added automatically as a detected search engine

Actual result:

Nothing happens

Expected result:

translate.google.com should be added to an "Other search engines" list on the page. An example of what this looks like in Chrome:
screen shot 2017-09-08 at 4 27 39 pm

Will the steps above reproduce in a fresh profile? If not what other info can be added?

Is this an issue in the currently released version?

Yes

Can this issue be consistently reproduced?

Yes

@bsclifton bsclifton added needs-mockup A feature which needs design mockup to be implemented. feature/search labels Sep 8, 2017
@da2x
Copy link
Contributor

da2x commented Sep 9, 2017

Please pay special notice to the drive-by search provider installation protection feature which work slightly differently in Safari and Edge. Safari also has a fallback for websites that don’t have an OpenSearch Description file (described in the previous link) which can be a … good source for inspiration.

PS: Chrome’s UI is the worst out of the lot here. Look at Edge and Safari instead. Firefox is more discovererable than the rest, but requires a separate search field/icon.

@Ornataweaver
Copy link

If you guys can't give a simple important feature after so long, you shouldn't make a browser, make a brave extension.
An ad-blocker which can pay the parties.
Although I understand you are renovating browsers, yet there are many extensions I need:
Stylish, OpenSearch, Domain Blocker, Div Blocker, etc.

@bsclifton
Copy link
Member Author

cc: @davidtemkin @zentagonist for prioritization

@cezaraugusto cezaraugusto added this to the Triage Backlog milestone Nov 8, 2017
@eljuno
Copy link
Contributor

eljuno commented Dec 3, 2017

Related to #10675?

@tildelowengrimm
Copy link

If OpenSearch is added, I'd love to be able disable it. I understand that it can be useful for folks who don't want to spend the time carefully wrangling their own custom search operators and keywords. I like to meticulously curate those items, and find the automatic addition of other search engines frustrating. Default-on with a settings toggle to disable, IMO.

@Lythimus
Copy link

Lythimus commented Jan 22, 2018

Brave nearly meets my browser needs, but it's frustrating that I can see the search engine list with shortcuts, but can't add my own. I've used this feature in Opera for like 10 years, so I may have to switch back to it if I can't figure out how to add searches through config files or something.

Edit: Updating lib.devTools.js, app.entry.js, and aboutPages.entry.js (and their corresponding map files) visually adds any searches you modify or add to the search config page, but it isn't hooked into the back end, so typing it in the address bar or setting it as default search won't have any effect even after reboot. So it seems there is not a config file which can be updated.

@bsclifton bsclifton added wontfix fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. labels Aug 25, 2018
@bsclifton
Copy link
Member Author

Happy to report this is fixed with brave-core 😄
screen shot 2018-08-24 at 10 56 29 pm

@bsclifton bsclifton removed this from the Triage Backlog milestone Aug 25, 2018
@tildelowengrimm
Copy link

But what if we added an "off" switch? brave/brave-browser#867

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature/search fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. needs-mockup A feature which needs design mockup to be implemented. settings wontfix
Projects
None yet
Development

No branches or pull requests

7 participants