-
Notifications
You must be signed in to change notification settings - Fork 470
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
Include search plugins favicons in this repository #42
Comments
In case of help this site will let you download the favicon of the site you want: http://www.getfavicon.org/ |
1337x @ngosang I couldn't find EZTV's favicon also because it's not accessible from my country (I proposed to remove it from the official list anyway). I think we should include these 16x16 pictures in qBt itself, and then tell the program to look after some local files rather than website links. |
I agree but It's a bit complicate. The code to download the favicons is in C++, not in this repository, and the C++ developers are busy with more important thinks. They have in mind to rewrite the search engine. I hope they have time for this when the time comes. |
The official plugin Zooqle doesn't have favicon and looks weird in qBittorrent.
I think would be a good idea to include the favicons in this repository and make a little change in qBittorrrent code so it tries to download from here first and, if not found, the current behavior.
What do you think @Chocobo1 @sledgehammer999 @Piccirello @zeule @glassez?
The text was updated successfully, but these errors were encountered: