Adam

like the design, simple and neat. the banner seems to be a bit low quality in some places but does fit in. I'd swap the "advertisers" and "login" boxes around. Not too keen on the altering background colour on the product review pages, reckon it should be a little lighter. Apart from them all good! Adam

you can't do it without slicing up the image. But if you want to make a decent website you're going to need to slice up that image and layout the page with divs and smaller, more usable images. Apart from that, once you have some content and detail on the pages I think it'll look pretty good! Though the text on the nav doesn't fit much at all, i don't know if you're planning to change it? Adam

It can definitely be done with PHP .. not too sure how though.

if ($result) { echo "Successful"; echo "<BR>"; echo "<a href='form.php'>Back to main page</a>"; } else { echo "ERROR"; }

157 views and no replies?

I've been developing this site for some time now. At this stage I'm not going to create a whole new design, but if anybody can give some suggestions on parts of it they think could be improved, that would be very helpful. Could you please also tell me your thoughts about the design, structure, etc. And if anybody finds any exploits could you please PM me with them. www.minimerc.co.uk Cheers, Adam

if (??? == substr($str,0,3)) $str's value will not change after using substr() unless you set $str as substr(...) : $str = substr($str,0,3); - Adam

I imagine you'd have to look into using "JOIN" .. to join the select and the count queries.. then i reckon you might be able to order them by the count. Search google for "JOIN". - Adam

code isnt that great.. been messing with for hours... <div id="wrapper"> <div id="header"><img src="imgs/adams-work.gif" alt="Adam's Work" /></div> <div id="tborder"><img src="img/top-border.gif" alt="" width="774" height="10" /></div> <div id="main"> <div id="lborder"><img src="img/left-border.gif" alt="" width="9" height="287" /></div> <div id="content"> </div> <div id="rborder"><img src="img/right-border.gif" alt="" width="10" height="287" /></div> </div> <div id="bborder"><img src="img/bottom-border.gif" alt="" width="774" height="10" /></div> <div id="footer"><img src="imgs/designed-by.gif" alt="Designed by Adam Boxall" /></div> </div> CSS #wrapper{ width:774px; height:auto; margin:20px auto; text-align:left;} #main{ clear:both;} #lborder{ width:9px; float:left; background:url('img/left-border-bg.gif') repeat-y;} #rborder{ width:10px; float:left; background:url('img/right-border-bg.gif') repeat-y;} #content{ width:755px; float:left; background:#FFFFFF;} td.main{ width:755px; background:#FFFFFF;} - cheers

Hi all, im trying to recreate my portfolio and make it validate as XHTML without any layout problems... At the minute I have a problem in FF where its adding padding to my div which contains an img. In the images attached you can see the difference between FF and IE, and Iv added a blue border to make it obvious. Iv tried adding margin:0px and padding:0px to the div container.. even to the image itself.. except it still just wont get rid of this extra padding in FF, and I don't know why its there.. - Can anyone help? Cheers, Adam [attachment deleted by admin]

it looks really rough.. not keen on the fonts, big gap under the nav, header doesn't look very impressive and there's some padding problems when you view it with firefox... lacks detail.

i think the navigation could be a bit better.. the links aren't really appealing. Really could do with some content, couldn't you make up a little "lorem ipsum" mock page? Would give a better idea on how its going to look. -adam

The headers nice, but im not keen on the marquee below it. The shoutbox looks too out of place, I think it would look more at home at the side of the content, or perhaps on its own page. Other than that I think you've done well.. the colours and the little menus all fit well. -Adam

I think the images are a little too blury, and the colours seem a little bland. The website doesn't really catch your eye. Id say perhaps create a better nav menu, use a variety of text sizes and try to create a header a little sharper and more eye catching.

I agree with the film god, I cant see a reason why the site should be aimed at Christians, its not like somebody will type into google "christian linux help".

Try not to use GETYOURLINKON.NET too much, it makes the text harder and longer to read... you can just use "we" or something. Btw this is the critique forum, for more system testing.. try the BETA testing forum. I had a quick look though and security seems alreyt! but im not an expert with the sql injections and that. All seems to run smooth enough, not keen on the like confirmed registration messages and that on a seperate page tho.. seems to take too long, best having them just appear on the homepage or the apropriate page or something. adam

freakstyle: I cant imagine creating another variable, then having the extra line of code to return the variable, would be cleaner / more maintainable than just using 'return true/false' as it happens? it's pretty self explanitory what's happening...

everything they've said, plus, i'd say make the "proceed to order" image rounded to fit the rest of the site.

im affriad i have to agree with the others, the design looks like it was made with frontpage. the use of the <hr> tag (especially as it's not even styled) just looks unprofessional. I'd say try and improve the graphics, perhaps make the font a little smaller, get the nav links on one line, change the purple background as the colors just don't go well together, create a better header because.. i don't even know what it is.. also i'd say the image that changes on each page should be made more seperate to the text.. perhaps a light border and a little more spacing?

filmgod, i think the idea of his 'personal website' is to act as portfolio

if you google it you'll find probably millions .. why not just invest in some web space though? it's not that much these days and you'll probably get the domain thrown in for free..

think your problem is in your CSS .. #menu a { ... height: 46px; ... } try putting the image on the outside of the link and see if that makes a difference..

hmm, the link seems to be redirecting me to techbrainy.com? i can't see that being the site you mean.. -cheers

the clips are now cyclic, and automatic too.. making the font sizes larger and trying to get the doctype to work, but having few problems in IE (tut)... I saw someone post a link for a website that helps with problems in IE when trying to get it valid, but i can't find it again, could that person please post the link again? -Cheeeeers!

how are you running it? (you said: "If i just run the function display(5); it works") .. does it append that HTML to the page's HTML?