...

View Full Version : Sites review



DooM_MonkeY
05-29-2004, 07:01 AM
i am a web design addict and i might need to start going to coders anonymous (lol) and i have made lots of sites for my friends, the only problem is im in the middle of making 4 sites at once, so all i want from you is to criticize THE TEMPLATE ONLY. None of the sites have much content but i wanted to see what all of you thought about them.

My Friends Starcraft/Warcraft site (games) (http://www.freewebs.com/angels-of-chaos) (My friend thought of the name, kinda corny)

My site (http://www.freewebs.com/Death_by_monkeys) i havent got around to making all the "position:absolute;" tags into a stationary table

Friend's site (http://www.freewebs.com/azianracer/) it is about a hobby called modifiers, you take toy cars and make custom stuff for them

and i have some more but am not yet ready to release them..... remember ONLY THE TEMPLATE lol....... o and if it changes matters at all, im 13 years old (trying to get ur sympathy lol jk)

1 last question, i wanna see who the youngest one here is? cause i have a friend that is not as good as me (not very good at all lol) but is 13 too and makes sites such as this (http://www.freewebs.com/muta-darkangels/)

ok, long message but im finally done

oracleguy
05-29-2004, 09:48 AM
My Friends Starcraft/Warcraft site (games) (http://www.freewebs.com/angels-of-chaos) (My friend thought of the name, kinda corny)

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 (http://www.codingforums.com/showthread.php?t=18346). 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 (http://www.codingforums.com/showthread.php?p=181585#post181585). Additionally, if you'd like some real world examples of site's that CSS based layouts, click here (http://www.codingforums.com/showthread.php?t=23475). 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 (http://www.w3schools.com/css/) especially.

Oh yeah, and you aren't the youngest around here... I think the youngest person on here is 11.

DooM_MonkeY
05-29-2004, 06:29 PM
Thank you very much, that one actually is one of my older sites, one of the newer ones is AzianRacer one, ive really been working on css lately and ive gotten better in that part. And about the frames, i only made those for there was way to much stuff to put on every page, that is why i use frames because of if I make an ajustment to one page, i have to do it to all, so yea, anyway what is about this thing only I can see? I didnt really understand that.......

Thanks again

oracleguy
05-29-2004, 06:38 PM
And about the frames, i only made those for there was way to much stuff to put on every page, that is why i use frames because of if I make an ajustment to one page, i have to do it to all, so yea

Then like I said, you might want to use
server-side includes (http://www.google.com/search?q=server+side+includes&sourceid=mozilla-search&start=0&start=0&ie=utf-8&oe=utf-8) to replicate that functionality of frames... without the frames. :)


anyway what is about this thing only I can see? I didnt really understand that.......



<!-- ZoneLabs Privacy Insertion -->
<script language='javascript' src='http://127.0.0.1:1027/js.cgi?pcaw&r=5537'></script>


Is what I'm talking about.

DooM_MonkeY
05-29-2004, 07:44 PM
<!-- ZoneLabs Privacy Insertion -->
<script language='javascript' src='http://127.0.0.1:1027/js.cgi?pcaw&r=5537'></script>

Is what I'm talking about.


I didnt put that there, whenever i make a page Zonealarm puts it there



EZ Archive Ads Plugin for vBulletin Copyright 2006 Computer Help Forum