Forum:Elinks-NPC updates

Forums: Village pump → Elinks-NPC updates
(This topic is archived. Please do not edit this page!)

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 <ul> 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_d³ 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. --Pcj (TDrop me a line!•C207,729 contributions and counting) 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. Sssssssssssssssssssssssss For Pony! (^Sssss/_Slithered) 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-<foo> templates to this style if nobody objects. Browsershots came back clean from what I can tell ([1]).

Oh, happy new year's. I'll be back in a few hours. --k_d³

Can we space them out a bit more? Especially with the Armory icon, it is nearly colliding with Wowhead. --Pcj (TDrop me a line!•C207,729 contributions and counting) 03:52, January 1, 2010 (UTC)

Sure. --k_d³ 06:39, January 1, 2010 (UTC)