Skip to content

Loading…

Adding missing search engine favicons #1350

Closed
halfdan opened this Issue · 13 comments

3 participants

@halfdan
Piwik Open Source Analytics member

Collecting missing search engine favicons here.
Keywords: favicon

@halfdan
Piwik Open Source Analytics member
@halfdan
Piwik Open Source Analytics member
@halfdan
Piwik Open Source Analytics member

Attachment:
hooseek.com.png

@robocoder

halfdan: will you be able to go through the entire list (core/DataFiles/SearchEngines.php and plugins/Referers/images/searchEngines)?

p.s. Looks like mserv.rrzn.uni-hannover.de.png is deprecated by above and can be removed.

@mattab
Piwik Open Source Analytics member

we don't need to put icons from the full list, as most search engines are very rarely used, and this would increase the archive size.

@halfdan
Piwik Open Source Analytics member

matt: alright
vipsoft: Sure, I'd just write a script to fetch all favicons.

The icons above are just those that were missing in my/the piwik demo stats.

@robocoder

I count ~230 entries in core/DataFiles/SearchEngines.php, and ~195 files in plugins/Referers/images/searchEngines. So, about 35 missing icons. If the average icon is .5K, I think we can afford an extra 18K added to the archive size.

(BTW we can free up 464K by not including "libs/jquery/original lib" in the archive.)

@mattab
Piwik Open Source Analytics member

Anthon, good point regarding "libs/jquery/original lib"; can you delete it and add in a REAMDE file what the files are (if there are special things to know about them)

Also OK I didn't realize that most icons were already in there.

@halfdan
Piwik Open Source Analytics member

Alright, assign me this ticket - I'll add the missing favicons.

@halfdan
Piwik Open Source Analytics member

Just to be sure: The favicon name is the url + .png of the first appearance of a search engine in core/DataFiles/SearchEngines.php?

In that case I have 82 new favicons and a list of 36 favicons that can be removed because of redundancy.

@halfdan
Piwik Open Source Analytics member

Attachment: Script to fetch missing and list not needed favicons.
ProcessFavicons.zip

@halfdan
Piwik Open Source Analytics member

Attachment: Favicons + List of not needed favicons
Favicons.zip

@robocoder

(In [2190]) fixes #1350 - add missing icons and remove redundant icons; contributed by halfdan; p.s. nice script

@halfdan halfdan added this to the Piwik 0.6.2 milestone
@halfdan halfdan self-assigned this
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.