Skip to content
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

Search and site suggestions does not work on duckduckgo #19

Closed
JoeTvx opened this issue Dec 3, 2017 · 18 comments
Closed

Search and site suggestions does not work on duckduckgo #19

JoeTvx opened this issue Dec 3, 2017 · 18 comments
Assignees
Labels

Comments

@JoeTvx
Copy link

JoeTvx commented Dec 3, 2017

Even when I enable this in Settings. Is this feature even supported by Bromite? If not, my apologies.

64.3278.0 on Android 8.0.0.

@csagan5
Copy link
Contributor

csagan5 commented Dec 3, 2017

Does it work with Google?

The URLs are specified: https://github.com/bromite/bromite/blob/master/patches/privacy/add-duckduckgo.patch

So it should work, unless some other patch disables them

@JoeTvx
Copy link
Author

JoeTvx commented Dec 4, 2017

Yes, suggestions work fine for both Google and Bing. Hmm, not sure why ddg is the exception.

I tried this in Chrome, and it works fine. I added the engine by going to duckduckgo.com and using their Add ddg to Chrome button. I can't do the same thing for bromite since the entry is already there by default.

Is there an easy way to see the suggestion URL configuration for and installed version of Bromite?

@JoeTvx
Copy link
Author

JoeTvx commented Dec 4, 2017

@csagan5, out anyone else, are you able to reproduce this?

@csagan5
Copy link
Contributor

csagan5 commented Dec 6, 2017

I figured out why. The URL specified there is incorrect (it comes from an original patch from Inox patchset if I remember correctly) and DuckDuckGo does not support suggestion URLs.

See https://duck.co/ideas/idea/547/query-suggestions-in-json-format-would-be-usuable

That is a question of ~5.5 years ago; I suggest using StartPage which is more transparent than DuckDuckGo, but it also does not have suggestions.

@JoeTvx
Copy link
Author

JoeTvx commented Dec 9, 2017

I do prefer startpage, but have not been using it because of the lack of suggestions support.

Is it easy at all for Bromite users to add the Google suggestions URL to the startpage search? In Chrome, duckduckgo.com can be added using their Add ddg to Chrome button, which sports suggestions.

@csagan5
Copy link
Contributor

csagan5 commented Dec 9, 2017

Startpage does not offer such feature...I suggest opening a discussion in their forums maybe?

Regarding DDG: can you open the development console and check what URL is being used for the suggestions printed in address bar? I have seen that they have suggestions on their home page, not from chrome address bar.

@csagan5 csagan5 closed this as completed Dec 16, 2017
@JoeTvx
Copy link
Author

JoeTvx commented Dec 22, 2017

Suggestions on startpage works fine on the Brave browser. I'm guessing the suggestions URL is separate from the search engine. In other words, they use Google for suggestions even on the startpage search.

@csagan5
Copy link
Contributor

csagan5 commented Dec 22, 2017

@JoeTvx I went to check in their source code: https://github.com/brave/browser-android-tabs/blob/master/components/search_engines/prepopulated_engines.json#L122

So now I will try using the same definition for DDG, it should work. Thanks for reporting an example of another Chromium-based browser (Brave) where it works.

By the way, I have seen that one can just visit a search engine page and then it is possible to add that browser. This feature must have been added recently in Chromium.

@csagan5 csagan5 reopened this Dec 22, 2017
@JoeTvx
Copy link
Author

JoeTvx commented Dec 22, 2017

Awesome, thank you. If you could also pick up the startpage definitions from Brave, that would be very helpful too since suggestions work on that as well.

Yes, I noticed the feature you're talking about as well, and would be very glad if it did indeed come from Chromium.

OT, but thank you so much for Bromite! The world sorely needs a modern open source browser with no strings attached!

@csagan5
Copy link
Contributor

csagan5 commented Dec 23, 2017

@JoeTvx yes it seems that they found some undocumented API, I will use their definition for StartPage as well. The next release is building so should be available in a day or so.

@csagan5 csagan5 added the bug label Dec 23, 2017
@csagan5 csagan5 self-assigned this Dec 23, 2017
@csagan5 csagan5 closed this as completed Dec 24, 2017
@JoeTvx
Copy link
Author

JoeTvx commented Dec 27, 2017

DDG suggestions work now, thank you!

Startpage suggestions don't work though. Nor can I add custom engines. Are there plans to include the latter feature?

@csagan5
Copy link
Contributor

csagan5 commented Dec 27, 2017

@JoeTvx adding custom engines is a feature of Chrome/Chromium, not Bromite. it works so that after visiting a search engine, you can go to the search engines selection in settings and pick it up

@JoeTvx
Copy link
Author

JoeTvx commented Dec 27, 2017

@csagan5, right, I understand, and hence my question. I see that feature in Chromium but not in the latest version of Bromite, and was wondering if you knew why.

@JoeTvx
Copy link
Author

JoeTvx commented Dec 27, 2017

@csagan5, do startpage suggestions work for you?

@csagan5
Copy link
Contributor

csagan5 commented Dec 27, 2017

@JoeTvx yes. you have to enable site suggestions because they are disabled by default in Bromite

@JoeTvx
Copy link
Author

JoeTvx commented Dec 28, 2017

I do have site suggestions enabled. It works fine for me in all other search engines except startpage. What gives?

@JoeTvx
Copy link
Author

JoeTvx commented Jan 1, 2018

Anyone else seeing this with startpage?

@JoeTvx
Copy link
Author

JoeTvx commented Jan 11, 2018

@csagan5, are you able to reproduce this? Should I open a separate ticket for this? Thanks.

ChanningPromethean pushed a commit to PrometheanWorld/bromite that referenced this issue May 12, 2023
…and AP9 Premium Chromium App

Merge in AP/chromiumpatches from AP9-3128 to master

* commit 'af4b2e1935195bde4ea152969fa6a928a215a6ed':
  AP9-3128 Implement telemetry events for AP9 and AP9 Premium Chromium App
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants