Site Reviews

Here are a few sites that I wouldn't mind you taking a look at and giving me some constructive critisicm or complimets. :cool:

http://www.dniwebdesign.ca.tt
http://www3.sk.sympatico.ca/trucking/
http://www.tritownthunder.com
http://www.nsjhl.com
http://www.crhswildcats.com
http://www.ncdgames.com
http://www.akennstrucking.ca
http://www.finalwarning.ca
http://www.usask.ca/goldenkey/
http://www.mcjannet.ca.tt

(These are two that I work on, Nipawin CU isn't updated yet and Jay Semko hasn't requested an upgraded in his design yet, but comments are still okay)

http://www.nipawincu.com
- my version http://www.dni-server.no-ip.com/nipawincu/
http://www.jaysemko.com

Some Other Sites I work on:
http://www.dni-server.no-ip.com
http://www.thepikes.ca.tt
http://www.damatgaming.ca.tt
http://www.thunderfm.ca.tt
http://www.ewatson.ca.tt (Just changed servers, some things may not work)
http://www.theciic.ca.tt


Thanks...

:eek::eek::eek:
Holy Jeeebus, you can't seriously expect us to review all those??!!

You don't have to do all of them... Even some is okay... Hard to think I still have time for a personal life and University at that...

(Almost done my 1st year, woooowhooooo!) :thumbsup:

Simply, your code style is not up to scratch. In terms of design, some of the sites are fine to an extent. However, one of your sites, you had a Flash intro - 800KB+, and it wasn't much at all. For dial-up users that would be hell. Granted you allow the user to decide to skip the intro, but those who don't have no choice but to wait - there are no links on the intro page to allow the user to follow through into the main page.

In terms of code, there is so much that you lack. Your own home page, for example lacks a DOC-Type, Meta Content Types, and structure. You use frames, where you have no need to, and you have used the <style> attributes in your HTML files, instead of using an external stylesheet. Further you state font size, not wise.

A lot of your code can be significantly reduced in size, you have a lot of pointless attributes:


A:link { text-decoration:none }
A:visited { text-decoration:none }
A:active { text-decoration:none }
A:hover { text-decoration:underline; color:#0066FF; }
That could be simply stated as:


a {
text-decoration: none;
}
a:hover {
text-decoration: underline;
color: #0066FF;
}
Another line of code that is completely useless is:


<LINK rel="SHORTCUT ICON" href="">

You have used that, although it would make no difference if the line weren't there at all, seeing as you haven't stated an icon to use. They are just a few of many other possible examples.

In my opinion, like many others, you have a lot to learn. For a first-year, it is good, but you really need to taken into consideration code standards.

The first thing I done was to look at your own website - but before I looked around that I looked at the source code. It is what I do for any site review that I come across. It provides a look into the mind behind the page.

I've seen that you have so much to learn, and you have some way to go.

In terms of design, your sites show the same kind of style, despite them having been from a variety of subjects.

I won't say more, but I'll simply recommend you have a look at www.w3schools.com and make reference to some of the topics in the HTML & CSS forum.