Forum:Elinks-NPC updates

So I'm skimming my talk page for things I've missed... Why are we linking to the armory for most NPCs? Only bosses return results. I've implemented elinks-NPC/dev. The notable changes are the addition of the |armory=1 tag to explicitly link to the armory (which can be bot-added thanks to the bosses cat) and using an &lt;ul&gt; instead of line breaks for accessibility purposes. Armory links also show links to the eu armory too, keeping in line with the rest of our elinks templates.

Thoughts? -- k_d3 02:11, January 1, 2010 (UTC)
 * I don't know, a consistent look? Would be nice if Armory were more full-featured, but it doesn't make much sense now to have an Armory link. -- 02:14, January 1, 2010 (UTC)


 * While being consistent is nice... I agree that unless a boss, its worthless to have shown. Its the same reason armory was removed from elinks-object... or rather never added. 02:48, January 1, 2010 (UTC)


 * Ok. Not that I figured making the armory links opt-in would be an issue. Thoughts on the ul/css? I'll update the rest of the elinks- templates to this style if nobody objects. Browsershots came back clean from what I can tell.
 * Oh, happy new year's. I'll be back in a few hours. -- k_d3
 * Can we space them out a bit more? Especially with the Armory icon, it is nearly colliding with Wowhead. -- 03:52, January 1, 2010 (UTC)
 * Sure. -- k_d3 06:39, January 1, 2010 (UTC)