Re: Another HTML II Student with Another Website

> I would like to submit my site for you all to have a look at.
> http://www21.brinkster.com/janettuggey/html2/diethome.html


Hello Janet,

First off I'll tell you that the website looked rather plain, reminiscent of
web pages 10 years ago. I recommend giving the site more style by changing the
default font and using more color (especially background color.)

Speaking of background color, it is important to specify the background color,
even if white. Earlier browsers (up to IE3 and NN3 I believe) have gray as the
default background color. Additionally, some people have changed their default
background color to another color of their choice. Always specify the colors
you want to use.

I'd be really surprised if you haven't heard by now (after completing HTML
II) that the filename of your "home" page should be "./", which you
should rename "diethome.html" to be. That way you wouldn't need to include the
filename in the URL, just http://www21.brinkster.com/janettuggey/html2/. As it
is now, I would normally be able to view all the contents of the directory.

I think the validation logos would look better centered on the page.

You have two @ in your email address on the home page.

On the home page you have a list of links with a description of each. On the
other pages you have only the links with no description. You should at least
give a description of the links via the 'title' attribute.

For the table at the top of the home page with "two graphic images either side
of the Page Heading", I recommend simply using TH for the middle cell instead
of TD, as the contents of TH cells are bold and centered. BTW, I don't think a
picture of cheeseburger is a good idea...

I suggest using the HTML element ADDRESS for your copyright notice. It is also
customary to include the copyright holder along with the notice, such as
"© 2003 Janet Tuggey. All Rights Reserved."

I suggest making the source code more legible by inserting blank lines between
paragraphs and the different sections of pages.


You use a lot of inline CSS styles which would be better put in an external
style sheet. You would link to the external style sheet by putting this LINK
in the HEAD section of every page (which is assuming you will name the style
sheet "diet.css"):

<link rel="stylesheet" type="text/css" href="diet.css">


body {color: black; background-color: white}

table.heading {width: 100%}
table.heading td {width: 15%}
table.heading th {width: 70%}

table.val-logos {width: 60%; margin: auto}
table.val-logos td {width: 33%}

h1, table.val-logos, address {text-align: center}


Using that style sheet, you can remove all of your inline styles. The table at
the top of your home page would need to be given class="heading", and the
table with the validation logos would need to be given class="val-logos". (Put
these attribute/value pairs inside the opening TABLE tag.)

I hope this helps! You are doing a good job. I am glad you are taking the time
to try to learn HTML right, and are concerned about having valid markup. Good
luck to you!


Tim
____________________________________________________________
http://webdesign.meetup.com/     See you at the next meetup!

HTML: hwg-critique mailing list archives, maintained by Webmaster @ IWA