Awhile back I wrote that I was having issues with Blogroll Links Favicons plugin for WordPress. I came up with some code that better displayed the favicons and fixed my issue. I wrote the author and he agreed that this code made more sense. I hadn’t thought about it, but recently I checked to see if the updated code had been put into the plugin and it hadn’t.
Just yesterday I emailed John about getting added as a contributor to the plugin, as he’s recently switched his personal site over to Posterous. He agreed to add me and I put my code in right away.
In the future I’m also going to be looking for ways of optimizing the plugin even more than it already is. I have quite a few links on any given page of my site that use this plugin, especially the about page. It’s getting to the point that having to retrieve all these thumbnails slows down my site, so I’m going to look into making this as fast as possible.
Let me know if you’ve got any questions or suggestions.
Hmmm, I just installed the new update and saw that the link text is showing up on top of the favicons. Seems like the text-indent is not working properly. I'm using Safari 4.0.4
Here is a pic:
http://elasticwax.com/images/favicons.tiff
This should be fixed now if you update to 1.6.1. The anchor links needed to be set to display: block to get the text-indent to apply.
Awesome! Thanks for the quick update.
Hi Josh,
I found it doesn't work perfactly in the wp2.9.1 default theme.
http://i46.tinypic.com/121wt3s.jpg
I think I've also fixed the issue you were having with this some time ago, but never got around to mentioning it on the blog. Let me know if you have any other issues.
Love your plugin. A cache function would be really nice. Is this something you are working on?
To be honest, I haven't done much with the plugin recently. I have some ideas on how I would like to implement a cache function, but haven't had time to really work on it at all. Hopefully I can find time to do this soon.
Josh,
I've been toying around with adding locally cached favicons; I don't think it would be too hard. I messed with it for a couple hours yesterday.
I wrote a function that takes in a favicon's uri and saves a local copy, but I can't get it to integrate.
Interested?
For sure! Email me the details and we'll figure it out. It would definitely help page load times.
is it possible to change code, to use it also for RSS feeds? let's say, that RSS feed includes links (from delicious, google reader or other sources) and I'd like to see favicons for all entries in feed…
You should look into SimplePie. I think there are some WordPress plugins out there for SimplePie.
HI there. The latest update is causing an error on my sites and not displaying the links at all:
Warning: parse_url() expects exactly 1 parameter, 2 given in URL/html/wp-content/plugins/blogroll-links-favicons/blogroll-links-favicons.php on line 296
Fatal error: Call to undefined function: scandir() in URL/html/wp-content/plugins/blogroll-links-favicons/blogroll-links-favicons.php on line 297
@Christian // This is definitely a problem with the version of PHP you have installed. Are you running PHP 5?
Try this for now:
•change line 296 to $linkdomain = parseurl($link);
•add this on a new line below it: $linkdomain = $linkdomain['host'];
Thanks so much Stephen! I changed PHP to PHP5 (should have been on that anyway) and it fixes it. 🙂 Thanks again for taking the time to reply with a solution.
@Christian // No problem man, don't hesitate to point out problems, inconsistencies, errors, or things you wish were different!
FYI, I had to make the following patches to 2.0.4 to get it to work on my setup. (http://pastebin.com/5G0G6agg)
They include changes to make it work on PHP4, to make it work on Windows (local test server), and to increase the timeout a bit (useful when there's a slow link to requery). Not in that order. 🙂
Silly link mangling. Try: http://pastebin.com/5G0G6agg