This time its for www.businessadvising. ca

Almost ready to go 100% live, but we need to know what needs fixing.

thanks

Well, I certainly like the look better than that last website you posted. There are a few things I'm not really liking about it though. Please take these as constructive criticism .

The first thing I don't like much is the font. If I were you, I'd go with a san-serif font. Times-New-Roman, or whatever that is there looks too default/noob. I would recommend something like arial (or something similar to it).

The next thing I don't like is that some of your images are warped. Try to crop all your images to the same size to prevent them from being stretched, it doesn't look very good. Also, try to make them all the same resolution.

Next, I don't know if this is possible, but maybe try making that flash graphic at the top span the width of the area taken by the text text. It looks kind of weird with those gray boxes on the sides.

Aside from that, not bad. I like how you used divs and style classes instead of tables, like some guy before said. That's how the pros do it . If I weren't way too lazy, I'd learn how to do that too, lol.

Although it's not feedback-y, this article may be a good read for you: http://talentegg.ca/incubator/2009/0...web-developer/

Originally Posted by blackdragon

This time its for www.businessadvising. ca

Almost ready to go 100% live, but we need to know what needs fixing.

thanks

You're improving : )

Your coding still really needs work. There are a lot of things in your css that are redundant and redundancy = slower loading time.

Don't use times new roman, it looks bad. Stick with a serif font like arial or verdana.

The header image looks like it's too narrow with those two grey blocks at the side.
"Home | Services | About | FAQ's | Testimonials | Contact | should be centered So should the bottom navigation. Also try to have it fit on one line, it's bugging me right now =p.

<style type="text/css">
<!--
body,td,th {


}
body {
background-color: #ececec;
background-image: url();
background-repeat: repeat-x;
color: #FFF;
}
.topHeader {
background-color: #CFF;
height: 20px;
width: 980px;
margin-top: 0px;
margin-right: auto;
margin-bottom: 0px;
margin-left: auto;
text-align: center;
color: #009;
}
.head1 {
color: #039;
}
.topHeader {
color: #006; (why is this here twice under the same attribute?)
}

.topHeader {
background-color: #ebf3fb;
(why is this here twice under the same attribute?)
}
.topHeader {
background-color: #3279b4;
(why is this here twice under the same attribute?)
color: #FFF;
}
.bannerHeader {
width: 980px;
margin-top: 0px;
margin-right: auto;
margin-bottom: 0px;
margin-left: auto;
text-align: center;
background-image: url(Images/pttrn.gif);
background-repeat: repeat;
height: 179px;
}



}
.navBar {
background-color: #e5e5e7;
background-repeat: repeat;
width: 980px;
margin-top: 0px;
margin-right: auto;
margin-bottom: 0px;
margin-left: auto;
color: #333;
}

.titleBar {
width: 978px;
margin-top: 0px;
margin-right: auto;
margin-bottom: 0px;
margin-left: auto;
background-color: #FFF;

}
.search {
text-align: right;
color: #000;
}
#logoBar {
text-align: left;
color: #003;
font-weight: bold;
}
#mainText {
background-image: url(Images/1-horizontal.png);
background-repeat: repeat;
width: 980px;
margin-top: 0px;
margin-right: auto;
margin-bottom: 0px;
margin-left: auto;
}
.subHeader {
text-align: left;
color: #D6D6D6;
font-family: "Palatino Linotype", "Lucida Sans", "Lucida Handwriting", Jokerman, Garamond, "Comic Sans MS";
font-weight: normal;
}
a:link {
color: #333;
text-decoration: none;
}
a:hover {
color: #03F;
text-decoration: none;
}

a:active {
text-decoration: none;
color: #0C9;
}
#mainBody p a:link {
color: #000080;
text-decoration: none;
}
#mainBody p a:hover {
color: #ff0000;
text-decoration: none;
}

#mainBody p a:active {
color: #ff0000;
}
#mainBody p a:visited {
text-decoration: none;
color: #000080;
text-align: right;
}
#mainBody {
width: 980px;
margin-top: 0px;
margin-right: auto;
margin-bottom: 0px;
margin-left: auto;
}

#logoBar p {
font-family: Arial, Helvetica, sans-serif;
text-align: right;
}
#logoBar p {
font-family: "MS Serif", "New York", serif;
}
#logoBar p {
font-family: "Times New Roman", Times, serif;
}
#logoBar p {
font-family: Verdana, Geneva, sans-serif;
}
#logoBar p {
font-family: Georgia, "Times New Roman", Times, serif;
}
#logoBar p {
font-family: "Courier New", Courier, monospace;
}
#logoBar p {
font-family: Arial, Helvetica, sans-serif;
}
#logoBar p {
font-family: "Palatino Linotype", "Book Antiqua", Palatino, serif;
text-align: left; (Why do you have logobar 123132 times with different attributes, unecessary)
}
.bannerHeader div {
font-family: Georgia, "Times New Roman", Times, serif;
font-size: 18px;
color: #009;
}
.homeMenuBar {
background-color: #CCC;
width: 980px;
margin-top: 0px;
margin-right: auto;
margin-bottom: 0px;
margin-left: auto;
text-align: center;
}
a:visited {
text-decoration: none;
color: #000;
text-align: left;
}
#mainBody {
background-color: #CCC;
color: #000;
background-image: url(Images/mainBack.jpg);
}
}
#footer {
width: 980px;
margin-top: 0px;
margin-right: auto;
margin-bottom: 0px;
margin-left: auto;
text-align: center;
background-color: #3279b4;

}
#mainBody {
background-color: #FFF;
background-repeat: repeat-x;
font-size: 18px;
background-repeat: repeat-x;
color: #000;

}
#mainBody p span {
float: left;
text-align: center;
}
#mainPara2 {
text-align: right;
}
#mainBody p {
font-family: "Times New Roman", Times, serif;
}

#logoBar {
border: thin solid #CCC;
color: #D6D6D6;
}

#mainBody p img {
padding: 10px;
float: right;
}
.finalline {
font-weight: bold;
}
#mainBody img {
float: left;
}
#mainBody p imga {
float: right;
}
-->
</style>

There's still lots of redundant things, but I need to go for dinner. I'll edit this later