Posted by Roi Danton on July 30, 2009 at 8:16pm
| Project: | Link |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Digidog |
| Status: | needs review |
Issue Summary
The filter module URL Icon appends the respective favicon to an external link. This is very helpful in link lists (e.g. reference lists or similar). It uses drupal_http_request() and stores favicons locally which decreases loading times for known sites. The code is in function _urlicon_format_favicon().
Comments
#1
How to I add my company's favicon to my URL
#2
You can achieve this effect using 4 lines of JavaScript as outlined in this blog post: Highlight external links with favicons
#3
But... what is it this ticket wants me to do? Okay, a module exists that will add favicons to links - that's good, I guess.
#4
E.g. add a new formatter which uses drupal_http_request() to retrieve the favicon of the linked site and displays the favicon next to the link.
However this request is so old, I have no more usage for it and so no time to provide a patch. Maybe the approach posted in #2 helps though the approach posted in initial post doesn't rely on external pages, so it don't depends on their links and their caches. I leave the issue status as is, you might want to close it.
#5
I'm not convinced this is something the link module needs to do, so I'm going to close it for now. If we suddenly come up with a lot of demand for it, we can revisit it.
#6
After seeing how Google automatically adds these, I stumbled across this issue. Here is a simple patch that adds an additional formatter, and uses the idea from #2 to automatically grab each url's favicon, and displays it next to the link.
#7
ha, 9 years ago http://drupal.org/node/632 ;-)
#8
@jhedstrom: thanks for the patch. can U please provide a patch for latest dev of D6 and D7 ? I would love to commit that to the final 7.x-1.0 release.
@bertboerland: jesus!
#9