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)

What links here
Dunno if it's a bug or known, but pages calling an image using this extensions don't show up on the "What links here" section of the description page. --Subfader 14:43, 3 April 2008 (UTC)
 * Correct. That's because this extension was specifically created to suppress links to the Image description page.  The "What links here" op on an Image's description page is not a list of where the image is used.  It's a list of pages that link to the Image's description page.  Tim Laqua talk 02:31, 4 April 2008 (UTC)
 * How could it be fixed? I have a similar extension written by someone else but atm I have the same problem. It works at least on Special:Whatlinkshere --Subfader 10:33, 6 April 2008 (UTC)
 * I can't think of a good way to do it without literally linking to the Images description page. You could check with the other extension's author and see if he is interested in implementing (I assume you're referring to ImageLink). Tim Laqua talk 13:24, 6 April 2008 (UTC)
 * Thanks, I got around it using a template which includes [[File:]] and the . This way the file link is called but hidden. --Subfader 15:15, 6 April 2008 (UTC)
 * Clever. ;-)  Tim Laqua talk 15:29, 6 April 2008 (UTC)
 * Implemented in r34776, v1.5. You shouldn't need the magic template anymore.  ;-)  Tim Laqua talk 01:57, 14 May 2008 (UTC)
 * Hehe cool. I tried and first I thought it doesn't work with but I quickly found out you shouldn't use underscores in the filename. it has to be the exact file name from the desc page (which never has underscores), so you cannot copy the destination filename on the uploading process. So if desc page is File:File Name.jpg
 * works / is used / linked
 * works (displays as intended) / but is not "used" (listed on Special:Unusedimages) and not listed on "what links here"...
 * I guess a fix is easy? --Subfader 11:26, 15 May 2008 (UTC)
 * Wierd. I'll take a look.  I used Title::getDBKey to get the name of the image page - sounded safe to me.  ;-)  Tim Laqua talk 21:56, 15 May 2008 (UTC)
 * I just tested and I can't reproduce. For me, it works with or without underscores.  Can you test again? Tim Laqua talk 22:07, 15 May 2008 (UTC)
 * Yeah sorry, you're right. Dunno what confused me, guess should have tested with new uploaded files. --Subfader 09:33, 16 May 2008 (UTC)