g1smd - 10:26 am on May 14, 2011 (gmt 0)
You do not need the keywords element. Searchengines don't use it.
Title element should be after the character set definition.
<font face...> and <font color...> is 1990s coding. You should be using CSS to set the style for the H1 element. The immediate problem is you open two of those <font> tags and only close one of them. You must close BOTH before you start the table.
Never link to "index.html". The canonical URL for the site root is "/".
You must have a space after an attribute; a="b"c="d"e="f" should be a="b" c="d" e="f" here.
Don't use <br> tags for creating space. Enclose the content in a heading, paragraph, list, table, form, or image tag and use CSS to set the paddding and margins.
You have a H1 element. The next one should be H2, not H4. Don't use the numbers to set the visible size. The numbers relate to IMPORTANCE. Use CSS to set the size of the H2 element.
Don't use XHTML style <br/> tags (those with the / in) in an HTML document; but do see the earlier note that says to not use <br> at all.
The <center> tag is in the wrong place. However, you should use CSS to style this. I haven't used a center tag since about 1998.
You are using a "partial DOCTYPE". Change the DOCTYPE to include the DTD URL, and browsers will render the page properly.
Put all your images in an /images/ folder then link to them using href="/images/image.png" with a leading slash in the link so that it counts from root.
Your coding style is more than 10 years out of date. I hope this isn't what someone is still teaching. They need a kick up the backside.