Template talk:Skillset/doc

Current issues I see: &mdash;Dr Ishmael 15:52, 4 June 2008 (UTC)
 * Need to add "vertical-align:top" to the style of the second row. You added that while I was typing this, it seems.
 * Incorporate the line break between Skill list items into that template, or do something else to hide unneeded line breaks. As it is now, there are eight line breaks in each table cell (except the first one, for some reason) regardless of how many SLIs actually output anything, forcing the cells to be eight lines in height.
 * Hah, I don't get an alert when you type here! I was still debugging and reading up on table tags,it should work now for 6 skill sets in a row, and it has no whitespace at either top or bottom. Now to make it gridify so that it displays two rows for 3 (or 4?) to 6 skill sets... --mendel 16:36, 4 June 2008 (UTC)
 * Addendum: You've probably seen on RC that I made a trim template to remove the line breaks and whitespace from start and end. It might come in handy elsewhere, too. --mendel 16:39, 4 June 2008 (UTC)

Can't get column/row borders to show up. They probably need to be defined in the style for td or be added to each td/th? --mendel 08:41, 5 June 2008 (UTC)

unneeded prof parm?
Since we've removed color from this, is the profession parameter still necessary? &mdash;Dr Ishmael 03:13, 12 June 2008 (UTC)


 * I've been debating this in my mind, too. As long as we keep it, we only need to change the template to put color back in - which we might decide after the Monaco move. If we skip it, the hack to name the unnamed parameters becomes more logical (starting with 1= instead of 2=). I could put code in to autodetect it: if parameter 1 is longer than 3 characters, it's a skill list, else it's a profession; and put up a hidden category if the profession is missing. I'd really like to ask b.r. about this as it's his project and he'll have to add the parameter - as it was his idea to have the colored version, I could imagine him wanting to keep the option for an easy switch. --mendel 10:31, 12 June 2008 (UTC)
 * The only other reason to remove the parameter is that it' snot so easy to put it in automatically, but in case we can't work that out (and it may be possible for the template to grab the profession off the monster using the BASEPAGENAME and my wikibase-fu} we can just leave it empty. Thoughts? --mendel 10:43, 12 June 2008 (UTC)
 * I'd recommend making the profession a named parameter to remove any possible confusion on parm numbering. If you name parms with numbers like "1=" etc., then give unnamed parms as well, the engine doesn't look at what names already exist before it numbers the unnamed ones, which means there'll be conflicts over which one actually comes through as  .  Naming the prof parm allows us to name the skill lists with numbers starting from 1 without any conflicts.  &mdash;Dr Ishmael [[Image:Diablo_the_chicken.gif]] 15:18, 12 June 2008 (UTC)