Thread history

From User talk:Zoycite
Viewing a history listing
Jump to: navigation, search
descTime User Activity Comment
07:41, 17 January 2013 Zoycite (Talk | contribs) New reply created (Reply to Some magic wiki trick for tables of unknown size ?)
14:59, 15 January 2013 Mandal (Talk | contribs) New reply created (Reply to Some magic wiki trick for tables of unknown size ?)
03:32, 11 January 2013 Zoycite (Talk | contribs) New reply created (Reply to Some magic wiki trick for tables of unknown size ?)
13:55, 9 January 2013 Mandal (Talk | contribs) New thread created  

Here is the template I used for tests : http://zoywiki.com/index.php?title=Template:LoH/CraftingItemInfoTest&action=edit

Mandal (talk)13:55, 9 January 2013

To me it looks like you just forgot to declare table rows in the top template. notice the pipe after the table attributes, |. and then the row markers |- before and after the table header cells marked by !. This is the more correct way to specify a table.

{|class="sortable" width="100%" cellpadding="0" style="padding:3px; padding-bottom:0px; color: #{{LoH/TableOuterTextColor}}; background: #{{LoH/TableOuterColor}};  font: 14px, Calibri, Verdana, Arial, sans-serif; margin-bottom:1em;" align="center" |
|-
!align="center" width="20%"|Card
!align="center" width="10%"|Recipe Type
!align="center" width="70%"|Recipe
|-
{{ #if:{{{recipe1|}}} | {{LoH/ForgeRow|{{{recipe1}}}}} |}}
{{ #if:{{{recipe2|}}} | {{LoH/ForgeRow|{{{recipe2}}}}} |}}
{{ #if:{{{recipe3|}}} | {{LoH/ForgeRow|{{{recipe3}}}}} |}}
{{ #if:{{{recipe4|}}} | {{LoH/ForgeRow|{{{recipe4}}}}} |}}
{{ #if:{{{recipe5|}}} | {{LoH/ForgeRow|{{{recipe5}}}}} |}}
{{ #if:{{{recipe6|}}} | {{LoH/ForgeRow|{{{recipe6}}}}} |}}
{{LoH/ForgeEnd}}
|} 
Zoycite (talk)03:32, 11 January 2013
 

Haha, right I forgot. Thanks :) But still, the unused parameters are considered as empty rows in the table. It looks weird... check http://zoywiki.com/index.php/LoH/crafting/Upstarts_Photograph_test

Mandal (talk)14:59, 15 January 2013

ah I see the problem...

{{ #if:{{{recipe6|}}} | {{LoH/ForgeRow|{{{recipe6}}}}} |}}

On all of your recipe lines you are saying

if( recipe6 is set )
{
display ForgeRow recipe6
}
else
{
display blank line
}

This is a pretty trivial fix, it is just the way in which you have closed your #if parser function

Change:
{{ #if:{{{recipe6|}}} | {{LoH/ForgeRow|{{{recipe6}}}}} |}}
To:
{{ #if:{{{recipe6|}}} | {{LoH/ForgeRow|{{{recipe6}}}}} }}

Also it might help to declare this block in your table as its own cell

|-
{{ #if:{{{recipe1|}}} | {{LoH/ForgeRow|{{{recipe1}}}}} |}}
{{ #if:{{{recipe2|}}} | {{LoH/ForgeRow|{{{recipe2}}}}} |}}
{{ #if:{{{recipe3|}}} | {{LoH/ForgeRow|{{{recipe3}}}}} |}}
{{ #if:{{{recipe4|}}} | {{LoH/ForgeRow|{{{recipe4}}}}} |}}
{{ #if:{{{recipe5|}}} | {{LoH/ForgeRow|{{{recipe5}}}}} |}}
{{ #if:{{{recipe6|}}} | {{LoH/ForgeRow|{{{recipe6}}}}} |}}

Making that a sub table might help not sure though i havent looked into it too far.
Zoycite (talk)07:41, 17 January 2013