Extension talk:Icon

I get the error Warning: Missing argument 2 for eficon_languagegetmagic in /home/imsaw/public_html/mediawiki/extensions/Icon/Icon.php on line 38
 * What verson of MediaWiki are you running? That's a call from the 'LanguageGetMagic' hook.  Tim Laqua talk 01:00, 16 January 2008 (UTC)

Paragraph around the image
The extension is wonderful, but I ran into trouble. It generates a new paragraph around the image. That works fine in most cases, in this case though, it ruins the table I use it in. How can that be fixed? Also is there a way to make it more flexible (closer to the Image: functionality) where you can omit image alt text and width and only add the link? Maybe add a class to it as well to make it more easy to customize it for end users.
 * You can stop the paragraph tags by wrapping it in a div - the P tags come from the parser trying to render valid XHTML:   You can omit alt text and width by simply leaving them empty:  Tim Laqua talk 03:41, 27 January 2008 (UTC)


 * Thanks a ton! Any chance you can add the div into the extension itself so it would do that on it's own? --Simon Moon 18:12, 27 January 2008 (UTC)
 * Addition: if I omit the width, it says "Bad Width" in the resulting preview page. --Simon Moon 03:41, 28 January 2008 (UTC)
 * What version of the Extension are you using? 1.1?  And I can't add the DIV in as when the page is parsed, the #icon parser function is replaced by a unique marker like: UNIQ-XXXXXXXXXXX-QINU and then the HTML output of the function is put back in later - this is to prevent corruption of the output.  But then we have the problem of plain text - plain text can't exist in the body tag, it MUST be encapsulated - which is why the parser puts those P tags in there.  wrapping the parser function in a DIV tag makes it legal XHTML.  ;-)  Tim Laqua talk 12:55, 28 January 2008 (UTC)
 * The Bad Width message should only appear if you provide a width, but it is invalid. Like if you put a space in there or a character value.  The following should be valid in version 1.1:  Let me know if that's not working.  Tim Laqua talk 13:00, 28 January 2008 (UTC)

to external link support
It would be really nice to be able to link external pages too (http://, https:// )
 * Implemented in r30400. Don't post new versions, post patches off the current SVN revison.  The version you posted broke existing implementations by wrapping the output in a DIV.  The links now have the 'iconlink' class and the image tags now have the 'iconimg' class.  External links have the 'plainlinks' class to suppress the CSS3 external link icon.  Also - sign your comments. Tim Laqua talk 18:15, 1 February 2008 (UTC)