FANDOM


 
Line 13: Line 13:
 
Overall, very cool. I like the large icons, but I may have to get used to the small icons. I kind of find them distracting rather than helpful, but that also might be because it's Clothcraft and not a craft I'm more familiar with (and the line spacing is smaller in Monobook, which doesn't help).
 
Overall, very cool. I like the large icons, but I may have to get used to the small icons. I kind of find them distracting rather than helpful, but that also might be because it's Clothcraft and not a craft I'm more familiar with (and the line spacing is smaller in Monobook, which doesn't help).
   
Edit: What do you think about renaming {{tl|In}} to {{tl|Iin}} or {{tl|IIn}}?
+
Edit: Oh, and what do you think about renaming {{tl|In}} to {{tl|Iin}} or {{tl|IIn}}?

Latest revision as of 18:05, February 26, 2018

Hmm... That's an interesting way of using DynamicPageList. I've looked at it before as a possible replacement for the category tables, but I hadn't considered using it like LabeledSectionTransclusion. I also didn't know it was enabled on this wiki. When were all these extensions added and not used? I need to browse through Special:Version, apparently.

One thing that might be an issue is that Wikia recommends to avoid using DPL more than once or twice on a page, but I'm not sure if that applies to this kind of use or not. It's a single result, with an include similar to LST (which we're already using a lot at this point), which is very different from the usual use case. The only real difference from using LST in this case is that DPL parses the page looking for a template parameter, which may or may not be costly depending on how it retrieves that information. But, just going by how fast the query is (using %DPLTIME% in the resultsfooter), it looks like it's at least as fast as retrieving by labeled section, if not faster.

As a side note, there are two ways that you could use DPL to get the item image from a page:

  • | include = {Item Image}:1 (template param; includes just the file name param)
  • | include = Item Image (labeled section; includes the whole section like LST)

My other concern is that this is a lot of content/images to be adding to some already fairly large pages. We should probably do a larger test before rolling it out completely, just to make sure it doesn't cause any issues for slower machines/connections (or hit the server too hard with all those LST/DPL calls).

As far as rolling this out with a bot - it's possible, but it would have to be in certain contexts, like job pages or merchant pages. It could also be added to the recipe link template (with the "noicon" param), though that has the same concerns as the recipe pages, especially for items that are used in a lot of recipes.

Overall, very cool. I like the large icons, but I may have to get used to the small icons. I kind of find them distracting rather than helpful, but that also might be because it's Clothcraft and not a craft I'm more familiar with (and the line spacing is smaller in Monobook, which doesn't help).

Edit: Oh, and what do you think about renaming {{In}} to {{Iin}} or {{IIn}}?

Community content is available under CC-BY-SA unless otherwise noted.