Extension talk:FaviconLink

Here's a link to various Microsoft Office and Windows programs.--Hoggwild5 02:09, 18 February 2007 (UTC)

I added the XSS alert because this works:
 * "> alert('hello');

Angela 19:43, 25 April 2007 (UTC)

Some Improvements
There are two issues with the current version:
 * if the target site has no favicon, an ugly placeholder is shown
 * some sites use favicons larger than the usual 16x16 pixels, this may look bad especially in lists of links

Replacing return " ".$linkname.""; with $headers = @get_headers($favicon); if (preg_match("|200|", $headers[0])) { // check for the HTTP return code; 200 means 'OK' return ' '.$linkname.''; } else { return ''.$linkname.''; // return only the link } should solve these issues.

Roberthaenel 19:29, 4 March 2008 (UTC)