eggcob

Just do this for each of your categories. <?php if ($page_title == 'news') { $newsactive = '2'; } echo '<a href="index.php"><img src="images/news'.$newsactive.'.jpg" border="0" ></a>'; ?> <?php if ($page_title == 'about') { $aboutactive = '2'; } echo '<a href="index.php"><img src="images/about'.$aboutactive.'.jpg" border="0" ></a>'; ?> and so on

I'm another one who doesn't like splash screens. Your site has a function and a goal to perform, you want to perform this goal as quickly as possible in as least steps as possible. The splash screen just adds another step into this process. I like the copy on the splash screen, and maybe that should be included somewhere in the main site. On entering your site i takes a while to figure out what the site is for. This is easily solved by adding something like you have on your what we do page to the grey square above your menu. "We make ethical T-Shirts" nice and clear. Gives the user an instant idea as to what you are about. Also, the clothing page has no clothing, it has a lot of text the another button to click through to clothing. Another step in selling a tee. What we do page could do with a few block quotes, picking up on "We don't manufacture clothes in a sweatshop" and "fits your body like a hot glove" Very nice site

Why not use text for the text instead of gifs? So much easier to do, less code and search bots will be able to read it.

First thing i usually do is try echoing the sql, then copy it over and run it on your database directly, make sure theres no typos etc. From first glance, is your id column id or ID? You will want to validate that $HTTP_GET_VARS['id'] before you run the statement, your leaving yourself open for injection attacks.

i found this book very useful when i was starting out PHP and MySQL Web Development, 3rd Edition http://www.amazon.co.uk/PHP-MySQL-Web-Deve...0121&sr=8-4 Good luck!

sure, $data_clientraw[$clientraw_cpt] ends up being a simple string. Which can then be echo'd. The only only thing you might have trouble with is your using " instead of ' if it throws up an error, swap your " for '. like this. if (eregi('Knots', $types_clientraw[$clientraw_cpt])) {$data_clientraw[$clientraw_cpt] = $data_clientraw[$clientraw_cpt]. ' kts <br />('.sprintf(KTStoMS($data_clientraw[$clientraw_cpt])).' m/s)';} for an explanation that makes sense... http://www.weberdev.com/get_example-3750.html

That wont work anyway, line breaks are ignored by browsers. try the <br /> version and let us know how you get on

if (eregi('Knots', $types_clientraw[$clientraw_cpt])) {$data_clientraw[$clientraw_cpt] = $data_clientraw[$clientraw_cpt]. " kts<br />(".sprintf(KTStoMS($data_clientraw[$clientraw_cpt]))." m/s)";}

What does $data_clientraw[$clientraw_cpt] usually contain? try this: if (eregi('Knots', $types_clientraw[$clientraw_cpt])) {$data_clientraw[$clientraw_cpt] = $data_clientraw[$clientraw_cpt]. " kts " . nl2br(sprintf("(" . KTStoMS($data_clientraw[$clientraw_cpt])))." m/s)";} Sorry if its garbage, half asleep

phpThumb is your friend here http://phpthumb.sourceforge.net/ phpThumb can cache the images at different sizes so they are only generated once, keeping server load down. This gives you the flexibility of being able to change the thumb size on the fly. If you were to have it generate resized files at upload, then change your design to one that requires a thumb at 100px wide, you'd have alot of work to do to resize 1000's of images. The way you request the images is bascially: <img src="phpThumb.php?src=images/23423r23423.jpg&w=70" /> With w=70 being the width in pixels. For info on assigning a random filename see linky Have fun