Yep, good advice indeed (not mention will also make your code mo' XHTML complient when you close them tags up nice n' purdy ).haydxn wrote: nice and efficient site design!
just a few quick comments, based on my own personal style of coding - and thus not necessarily important to anyone else!
you use a <li> tag to make each link a 'list item'. that's fine, and you can see that it works well enough, but i feel that it is good practise to get into the habit of closing your tags. like the <a> tag gets closed with an </a>, the <li> should really get closed too with an </li>; this makes the page more logical, and can be a useful instinct as you get deeper into html. also, because the items are list items, it's also good to put them within a logical list; if you surround the list items with <ul> ... </ul>, you'll declare them to be an unordered (bulleted) list.
you can change that to an <ol> (ordered list) and then they'll magically be numbered!
as you've seen, it works and is therefore perhaps not too important for your needs. i just find it good to develop handy habits that can make future developments easier should you want to take things farther than you'd initially imagined.
btw i really hope this doesn't sound like i'm being unnecessarily critical! just providing some things that may help!
Another benefit of that will become more appearent once you dive into the fun of CSS: by having a complete tag-set to encapsulate stuff in, you can then easily attach CSS classes to them to globally control what they contain.
So for example, you could take those <li></li> tag-sets and make sure that all such list elements use the same font characteristics (instead of using messy FONT tags each time), like so in yer STYLE header:
Code: Select all
li {
font-family: arial,sans-serif;
font-size: 12px;
font-weight: bold;
color: #ff0000
}