This Forum has been archived

Visit the new Forums
Forums: Index WoWWiki technical 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 <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_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. --PcjWoWWiki admin (TDrop me a line!C58,926 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. User:Coobra/Sig4 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_d3
Can we space them out a bit more? Especially with the Armory icon, it is nearly colliding with Wowhead. --PcjWoWWiki admin (TDrop me a line!C58,926 contributions and counting) 03:52, January 1, 2010 (UTC)
Sure. --k_d3 06:39, January 1, 2010 (UTC)

Ad blocker interference detected!

Wikia is a free-to-use site that makes money from advertising. We have a modified experience for viewers using ad blockers

Wikia is not accessible if you’ve made further modifications. Remove the custom ad blocker rule(s) and the page will load as expected.