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)

Hi @Roberthaenel,

Great - your improvment code works! But sorry for asking - I do not understand what exactly should be shown/appear as result, IF the target site has no favicon (e.g. an icon so that all links and favicons are -vertically- in line).

I mean, if through your new code simply no favicon should appear in this case (because the site simply do not have a favicon), I think in this case I would simply not use the anyweb-extension(tags) OR would simple replace (instead a placeholder or whatever) through the the browsers "no favicon icon" or maybe any other icon. Therefore, it would be a nice next improvement, if your code would be able to automatically replace the "no icon place" (similar as browsers do so) with the browsers "no favicon icon" or a possibility to replace a given place through any other icon (or maybe a variation possibility for both?) ...

--80.109.228.11 17:08, 3 April 2008 (UTC)