Quote:
|
Originally Posted by DooM_MonkeY
|
I'll review this one for you.
I'll first start with the UI. Having the 3 iframes makes the page difficult to use, especially considering the main page itself scrolls. There is no real reason to use them, so I suggest you remove them. If you want to make it easy to update, I'd suggest server-side includes. The black space above the header doesn't work well, at the very least reduce the spacing. It also repeats horizontally for me (running 1152x864), you'll want to look into fixing that. I won't review it much farther since artistic stuff isn't my thing.
Now onto the code (which is my thing). Frankly, it needs a lot of work. First thing is that your missing a
doc-type. Once you select a doc-type it will determine how much of the code you can salvage, you are using a lot of older tags and attributes that have been deprecated. You are also using tables for layout purposes and
you really shouldn't do it. Additionally, if you'd like some real world examples of site's that CSS based layouts,
click here. One simple thing too, you have your style tag inside your body tag, that belongs inside your head tags but I see that you don't actually have head tags, which you'll want to add. Your title tag will go inside of it as well. You might also want to get rid of or fix that script tag that points to the localhost IP address since that won't do anything on any computers except yours.
You seem to have a somewhat good start for being 13 and with just a few more skills you should do pretty good. I suggest you
read up more on CSS especially.
Oh yeah, and you aren't the youngest around here... I think the youngest person on here is 11.
Before you post, read our: 
