Oh boy are you in for it now! Website, just looking at it - it seems your entire top div is a link back to the home page, but it is designed to look like it is in different pieces. You may not necessarily need to have the "home page" part of the picture up top. - you should put a text based shortcut menu in the footer that has Home | the other top level menu choices. - the contact us/courses/links boxes bottom borders align differently in relation to the line in the background image differently on each page. - the picture on the About Us page should have a background that matches the page content color. Same with the gear in the Courses box. In fact a lot of images are like this. - the social networking icons are larger than your top level menu options, meaning the viewers focus will gravitate towards them. - the three contact images on the Contact Us form do not match the overall color/design scheme of the site. - the hover color of the "head office" link on the bottom is too similar to the background color on the gradient. - Your copyright says 2010... - on some pages, the Links box at the bottom has a picture in it. Website, behind the scenes - Google Analytics code should be at the bottom of the document and not the top. You can find best use instructions on your GA account. - Tables are for tables not for layout! - you have mixed case in some attributes, and use full URI for some and not others. Example: <p><a href="www.hse.gov.uk" title="HSE">www.hse.gov.uk</a></p> <p><A href="http://www.ilcor.org/" target="_blank">www.ilcor.org/</A> - your CSS validates, which is great, but your HTML has no Doctype. - you are specifying attribute tags for objects that do not support them, such as Marginwidth, MarginHeight, etc. - Your code is closest to HTML 4.01 Transitional. Add this as your Doctype, and adjust your code appropriately if you want that part to be good. It is fine to make coding sacrifices of course, for example on my own site, I'm not overly concerned about having ALT tags for every image or specifying an encoding type. forum - the logo on your forum does not match your website. - the design and colorscheme of the forum does not match the website - the forum #1 user is Admin. This is bad. Always set to hide the Admin account. Then your second user is a Global Mod and is your everyday use account. - not sure why all the forum hovers say Merry Christmas on them... summary It looks fine and is likely to be functional for what it is designed for. Nothing really glaring except for the GA placement. I also didn't bother testing your contact form. You want to make sure that is sanitized and secure since it is a direct source into your webserver. In the end, this looks like a run of the mill business website, which isn't a knock against you since it is your first website. Heck my first website looked like everyone else's too. That's just how you start.