Some very quick points:
- Remove all the META tags except for Title, Description, Robots, Content-Type and maybe Language. All others serve no purpose anymore.
- Move the Styles (CSS) into an external stylesheet and call it from your pages. This way its cached on the users system and reduces the 'weight' of a page
- To align the text in a container, like a DIV use a Style and not the align property which has been deprecated. To do so create a CSS rule like:
.text-center {text-align:center}
and then call it from the container where the text should be centered like:
<div class="text-center" ...
If you want an img centered, without being contained within a container you can use a rule like:
.img-center {display: block; text-align: center;margin:0 auto;}
These are some of my base rules:
.clear { clear:both; }
.float-left { float:left; }
.float-right { float:right; }
.text-left { text-align:left; }
.text-right { text-align:right; }
.text-center { text-align:center; }
.text-justify { text-align:justify; }
.img-middle {vertical-align:middle}
.img-center { display: block; text-align: center;margin:0 auto;}
.img-left { float:left;margin:4px 10px 4px 0; }
.img-right { float:right;margin:4px 0 4px 10px; }
- Don't use Caps (or spaces) in any file name (including images names / folder names) or URL. It only leads to issues down the road and will lead to problems on a Linux server. Users are used to lowercase and type them by default.
- Validate the page through the W3C (
http://validator.w3....endsofgovi.org.uk%2F). Validation doesn't mean it will work in every browser or look the same but it helps. It also helps get rid of the tricky little displays bugs that may pop up.
- Close all you tags (i.e. <META NAME="Title" CONTENT="Friends of GOVI"> should be <META NAME="Title" CONTENT="Friends of GOVI" />)
- The sidebar would do better as an unordered list styled into a menu then a bunch of P tags. Its semantically more correct.
Thats it for now as I was just looking at the code.