Improved ThreadReview Template.

[Sinecure]

Note:I posted this on my own template modifications section (http://www.pixeljunction.com/forum/forumdisplay.php?s=&forumid=7) at Pixeljunction.com

Updated it to get ride of the Pixeljunction stuff in it :)

Basically this template modification will shorten how much code is used in your “Thread Review” and in addition to that it makes it look far better, by getting ride of the large “Gap” between each review. This gap is caused because they use a new table for each
instance of the review.

Here is a preview of what it looks like on Pixeljunction Version 3:


Here is a before shot of the “gaps” as I explained:


Here is what it will look like after:



You will need to modify the following templates:
newreply
threadreview
threadreviewbit


First template we are going to modify is the “New Reply” Template.
Most likely you may have modifications to your New Reply template, so I’ll post what you have to change.

In newreply find:


<!-- spacer -->
<table cellpadding="0" cellspacing="0" border="0" width="{tablewidth}" bgcolor="{pagebgcolor}"><tr>
<td><img src="{imagesfolder}/clear.gif" alt="" width="10" height="1"></td><td width="100%">
<!-- /spacer -->
<table cellpadding="{tableouterborderwidth}" cellspacing="0" border="0" bgcolor="{tablebordercolor}" {tableouterextra} width="{contenttablewidth}" align="center"><tr><td>
<table cellpadding="4" cellspacing="{tableinnerborderwidth}" border="0" {tableinnerextra} width="100%">
<tr>
<td bgcolor="{tableheadbgcolor}" colspan="2" align="center"><normalfont color="{tableheadtextcolor}"><b>Topic Review (Newest First)</b></normalfont></td>
</tr>
</table>
</td></tr></table>
<!-- spacer -->
</td><td><img src="{imagesfolder}/clear.gif" alt="" width="10" height="1"></td></tr></table>
<!-- /spacer -->
$threadreviewbits


Replace it with:


<!-- spacer -->
<table cellpadding="0" cellspacing="0" border="0" width="{tablewidth}" bgcolor="{pagebgcolor}"><tr>
<td><img src="{imagesfolder}/clear.gif" alt="" width="10" height="1"></td><td width="100%">
<!-- /spacer -->
<table cellpadding="{tableouterborderwidth}" cellspacing="0" border="0" bgcolor="{tablebordercolor}" {tableouterextra} width="{contenttablewidth}" align="center"><tr><td>
<table cellpadding="4" cellspacing="{tableinnerborderwidth}" border="0" {tableinnerextra} width="100%">
<tr>
<td bgcolor="{tableheadbgcolor}" colspan="2" align="center"><normalfont color="{tableheadtextcolor}"><b>Topic Review (Newest First)</b></normalfont></td>
</tr>
$threadreviewbits



Replace your “Threadreview” template with the following:


<tr id="cat">
<td bgcolor="{tableheadbgcolor}" align="center" colspan="2"><normalfont color="{tableheadtextcolor}"><b>
This thread has more than $maxposts replies.
Click <a href="showthread.php?s=$session[sessionhash]&threadid=$threadid" target="_blank" class="thtcolor">here</a>
to review the whole thread.</b></normalfont></td>
</tr>
</table>
</td></tr></table>
<!-- spacer -->
</td><td><img src="{imagesfolder}/clear.gif" alt="" width="10" height="1"></td></tr></table>
<!-- /spacer -->



Replace your “Threadreviewbit” template with the following:


<tr>
<td bgcolor="$backcolor" valign="top" width="175" nowrap><normalfont>$username</normalfont></td>
<td bgcolor="$backcolor" valign="top" width="100%"><normalfont>$reviewmessage</normalfont></td>
</tr>


That’s it ,it looks much better in my opinion :)

[nuno]

Nice :)
What happens to text within [code] and [php] code tags?
Will it stretch all tables to the right? :confused:

[Mr. X]

Cool. I'll try this out later since I gotta go through all the code and remove the pixeljunction.com specific stuff.

[SaintDog]

Would you care if I added this to vBT (giving your credit of course)?

Regards,

- SaintDog

[N9ne]

Hey this is a nice mod :)

[Sinecure]

Originally posted by SaintDog
Would you care if I added this to vBT (giving your credit of course)?

Regards,

- SaintDog


knock yourself out, just link to Pixeljunction :)

[Sinecure]

Originally posted by Mr. X
Cool. I'll try this out later since I gotta go through all the code and remove the pixeljunction.com specific stuff.

Crap, its suppose to be {imagesfolder} , bah!

[SaintDog]

No problem, will do :).

Thanks,

- SaintDog

[Sinecure]

OK,I updated it so that none of the Pixeljunction stuff is in there anymore, should work perfectly :)

[Dean C]

thanks for this... :D

[Mr. X]

sweet, works great for me, thanks!

[Steve Machol]

BTW your links in the vB Borders Tutorial are broken:

http://www.pixeljunction.com/tutorials/html/vbborders/

[Sinecure]

eeek, thanks Steve :)