I feel stupid to ask

richardedmunds

$sql = mysql_query("SELECT * FROM images order By ID DESC LIMIT $from, $max_results");

while($row = mysql_fetch_array($sql)){

// Building my table to show images

echo "<center><table border = 2><TR><TD>
<h3><img src = \"".$row['link']."\" height = \"200\" width =\"200\"></td></tr></table>";
}

Hey guys,
Basically the above is my code and it displays an image loaction from a database table(text), eg "http://www.url/image01.jpg"
but obviously it is showing the images one on top of the other(as i end the TR and the TABLE), but i want to arrange it so its 3 images across. Have you ever had the feeling you know it and its on the tip of your tounge? well thats me, have racked my brains and nothing but i know its there.

njm

Richard,

Take the <table> and <tr> tags out of the loop.

echo '<table border=2><tr>';
while ($row = (mysql_fetch_array($sql))
  {
    echo '<td><img src = "'.$row['link'].'" height = "200" width ="200" /></td>';
  }
echo '</tr></table>';

this should put the images into three separate <td> cells in the same <tr> row.

hth, njm

scriptkiddie

Originally posted by njm
Richard,

Take the <table> and <tr> tags out of the loop.
echo '<table border=2><tr>';
while ($row = (mysql_fetch_array($sql))
  {
    echo '<td><img src = "'.$row['link'].'" height = "200" width ="200" /></td>';
  }
echo '</tr></table>';
this should put the images into three separate <td> cells in the same <tr> row.

hth, njm [/B]

Actually, njm, it won't. It will display all the images in the DB in one row.

Here is a suggestion how to to it:

<?
// Assign variables
$per_row = 3; // Number of images you want per row
$x = 0; // Counter variable

echo '<table border=2><tr>';
while ($row = (mysql_fetch_array($sql))
  {
    echo '<td><img src = "'.$row['link'].'" height = "200" width ="200" /></td>';
    if ( $x == (int)$per_row )
    {
        echo "</tr>";
        $x = 1;
    }
    else
    {
        $x++;
    }

  }
echo '</table>';
?>

This will put a </tr> on every third time the loop runs.

Hope this helps!

richardedmunds

the problem is that table could have many more items 🙁
currently there are 14 and its increasing all the time. means by taking the <TR>and <TABLE> out the images will all be aside another.

i want it to output like

[image1] [image2] [image3]
[image4] [image5] [image6]

Sorry for explaining it wrong.
but thanks for the speedy reply 😉

any ideas on how to do the above though would be great

richardedmunds

scriptkiddie

that looks better!
will give that a go and have my fingers crossed 😃

thanks both of you and will let you know how it went

richardedmunds

WORKING 🙂

THANKS

it looks good!!!!

made me happy now 😃

njm

Scriptkiddie,

Actually, njm, it won't. It will display all the images in the DB in one row.

this should put the images into three separate <td> cells in the same <tr> row

That's the same thing 😉 - although I read it as returning just those three images

njm

scriptkiddie

Richard:
Glad it worked out 🙂

njm:
Ok. Seems like we had different ways of understanding the problem. Your code will work just fine for 3 images.