-
Notifications
You must be signed in to change notification settings - Fork 164
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
Integration of Yandex search engine in the "search with" menu #217
Comments
@XengShi are we adding it? |
Yess, but snowing this to non-russian counties will feel irrelevant so we can write function to show only if the userCountry is Russia (or any other countries where people use yandex) or adding toggle to show/hide in settings page. |
to show only if the userCountry is Russia -- maybe this can be done since weatherAPI also returns country name when location is entered. But adding an option in menu seems better to me, where user can select which search engines to show. And we can also add Yahoo and Baidu, alongwith Yandex. Something like multi select picklist. Toggle only for Yandex doesn't seems good idea to me. |
Yess this would be great, you can contact the person who added the ability to edit and add more shortcuts. A similar UI would be nice for 'search engines' |
This would be great! |
I tried integrating it myself but failed to match the styling of the icons with yandex logo, if you would be kind enough to add this minor addition in the next release it would be awesome .
Great extension overall :)
The text was updated successfully, but these errors were encountered: