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

Unterstützung von mehr Search-Engines #28

Closed
C0ntroller opened this issue May 3, 2021 · 6 comments
Closed

Unterstützung von mehr Search-Engines #28

C0ntroller opened this issue May 3, 2021 · 6 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@C0ntroller
Copy link
Member

Derzeit wird nur die Google-Suche für die Weiterleitung unterstützt. Folgende Suchmaschinen wären ebenfalls wünschenswert, da nicht selten genutzt. Dahinter jeweils die patterns (statt keyword müsste dann bswp. "hisqis" stehen):

  • DuckDuckGo: https://duckduckgo.com/?*q=keyword*
  • Ecosia: https://www.ecosia.org/search?*q=keyword*
  • Bing (gerade für edge-Nutzer): https://www.bing.com/search?*q=keyword*
  • Startpage: https://www.startpage.com/do/search?*q=keyword*
  • Qwant: https://www.qwant.com/?*q=keyword*

Da ja eine Änderung des Manifest die Erweiterung deaktiviert, ist hier wahrscheinlich keine triviale Lösung möglich, sondern das muss ebenfalls in background.js angelegt und über einen Button erfragt werden.

@C0ntroller C0ntroller added the enhancement New feature or request label May 3, 2021
@OliEfr
Copy link
Member

OliEfr commented May 3, 2021

Hey @C0ntroller,

ja, das ist eine sehr gute Idee - ich nutzte selber teilweise DDG und ärgere mich jedesmal :D

Genau, dass stimmt. Wäre vermutlich eine optionale Berechtigung Zum einen kann man die in die bisherigen Anfragen einbinden. Ansonsten kann man das über ein notification banner in OPAL machen.

Ich hab übrigends schon drüber nachgedacht, eine Berechtigungen für alle Domains - also <all_urls> nachzufragen und die später fest mit ins manifest zu nehmen. Weil dieses Berechtigungsgehabe ist echt nervig - besonders für Leute, die Neuinstallieren.

EDIT: Also ich denke es passt einfach <all_urls> nachzufragen und gut ist.

@TUfast-TUD TUfast-TUD deleted a comment from C0ntroller May 13, 2021
@OliEfr OliEfr added the good first issue Good for newcomers label Jul 18, 2021
@OliEfr
Copy link
Member

OliEfr commented Aug 2, 2021

Update: die Nutzer wurden nun zweimal darauf hingewiesen, die Berechtigung für alle URLs zu akzeptieren.

Die automatische Weiterleitung für viele search-engines könnte nun implementiert werden. Außerdem können alle URLs als verpflichtende permission ':///*' ins manifest genommen werden.

@C0ntroller

@C0ntroller
Copy link
Member Author

Außerdem können alle URLs als verpflichtende permission :///* ins manifest genommen werden.

Ist das wirklich absolut fest? Dann würde ich das nämlich mal machen und evtl die Skripte, die derzeit im Code registriert werden direkt ins Manifest packen. Mehr search engines ist ja dann nur wenig Arbeit

@OliEfr
Copy link
Member

OliEfr commented Aug 16, 2021

Ja, das ist absolut fest. Wir verlieren wahrscheinlich ein paar Nutzer, aber den Wartungsaufwand macht das so viel einfacher! Vergiss nicht, dass die Berechtigung für alle URLs noch nicht im manifest ist.
Die optionalen Skripte kannste gerne mit rein machen. Danke dir!

Und wenn du Zeit hast: gerne TUfast-Camp vom 27. - 29.09. in DD ;)
Hier ein matrix-chat für einfachere Kommunikation: https://matrix.to/#/#tufast:fugi.dev Kannst über TU-Login beitreten.

@C0ntroller

@C0ntroller
Copy link
Member Author

Und wenn du Zeit hast: gerne TUfast-Camp vom 27. - 29.09. in DD ;)

Hab schon gesehen, aber leider bin ich da nicht da ^^ Ich hab einen PR reingestellt für "alle Permissions" und theoretisch alles für einen zweiten PR vorbereitet, um alle genannten SEs plus die neue von Brave zu unterstützen.

Ist nur oberflächlich getestet, sollte aber soweit klappen.

@OliEfr
Copy link
Member

OliEfr commented Aug 16, 2021

@C0ntroller habs gesehen - danke. Ist gemerged. Ab jetzt PR bitte auf develop-branch stellen.

Bitte sorgsam Testen. Das ist wichtig.
Oli

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants