Don't hate me...here's my suggestions
1) Your use of table's or layout is going to get a lot of heat around here. CSS is the best way to do it.
2) You sporadicly close out your table tags. Some have closing tags, some don't. You should be closing them all out.
3) Seems like a lot of coding being done for something that could be looped. I would create arrays for the sections and titles, and loop through them. Here's an example(working with your existing code)
PHP Code:
<?php
include_once "header.php";
include_once "config.php";
$sections = array("Article",
"Concert Review",
"Lyrics",
"Philosphy",
"Story/Poem",
"Tutorial");
$titles = array("Articles",
"Concert Reviews",
"Lyrics",
"Philosphies",
"Stories/Poems",
"Tutorials");
?>
<table border="0" width="100%" cellspacing="0" cellpadding="0">
<tr>
<td align="left"><b><?=$titles[$i]?></b></td>
<td align="right">
<a href="articles.php?by=desc"><img src="images/up.gif" border="0"'></a>
<a href="articles.php?by=asc"><img src="images/down.gif" border="0"></a>
</td>
</tr>
</table>
<?php
for($i=0;$i<count($sections);$i++) {
if(isset($Result)) {
unset($Result);
}
if(isset($row)) {
unset($row);
}
$Result = mysql_query("SELECT *
FROM ".$sqlTable."
WHERE article_sec='".$sections[$i]."'
ORDER BY article_title ".$by);
while ($row = mysql_fetch_object($Result)) {
?>
<table border="0" width="100%" cellspacing="0" cellpadding="0">
<tr>
<td align="left"><a href="article.php?id=<?=$row->id?>"><?=$row->article_title?></td>
</tr>
</table>
<?php
}
}
include_once "footer.php";
?>
HTH
Linear Mode
