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)

Inside an unordered list
I'm trying to use  inside an unordered list. Wikitext looks like this:
 * test item
 * other item

HTML output looks like this:

 test item   other item

Is there any way I can get the   inside the   ? Is it the parser trying to generate valid XHTML again? Jlerner 21:21, 11 February 2008 (UTC)
 * Yeah, it has to do with the order that the parser handles things. There 's a line break thrown in there at some point (hence, the DIV wrapper) - but since the MW style UL blocks are based on consecutive lines - it being wrapped in a DIV doesn't help there (because the block level processor finds a line that doesn't start with a * and closes out the list).  The only way to do it (afaik - i'm not a MW parser guru) is with an HTML list (in which case you don't need the DIV wrapper as it's wrapped by the LI tags:

 test item </li> other item</li> </ul> Usually fine f/ admins, but asking normal users to use HTML is not desirable Tim Laqua talk 01:40, 12 February 2008 (UTC)
 * Looks like changing the last line of function efIcon_Render</tt> to the following fixes the problem!

return $parser->insertStripItem($output, $parser->mStripState);
 * Credit goes to User:Jimbojw and his extension EmbedVideo 0.1.2. Jlerner 06:00, 12 February 2008 (UTC)
 * Interesting. I'll update the extension later today.  Thanks for investigating!  Tim Laqua talk 13:35, 12 February 2008 (UTC)