1 reply [Last post]
keyboard
Offline
newbie
uk
Last seen: 14 years 24 weeks ago
uk
Joined: 2005-10-13
Posts: 9
Points: 0

Hi iam kind of new to building websites this is my first CSS site...i did have this site in HTML using tables but i found CSS and got me there a lot easy...

http://www.freehtmlab.com/index.html

If the link dont work try just the domain instead of index.html

Some suggestions comments anything realy...

Iam still moving all my pages from HTML to CSS and XHTML so not all links and stuff will work but point me in the right direction LOL Laughing out loud

briski
briski's picture
Offline
Elder
London
Last seen: 9 years 25 weeks ago
London
Timezone: GMT+1
Joined: 2004-02-16
Posts: 1066
Points: 0

Site review....

Couple of quick points. Firstly, what borwsers have you tested this in. There are problems with the logo in both IE and Firefox (on PC) and the "vote for us" section looks off in IE and very odd in FF.

Also you have code like so

<center><hr size="1" width="400px" /></center>
<br />
<br />

Which whilst valid (although I could have sworn that <center> was not valid any more) is really not the best way to do thing's You should be using CSS to pad and center these things rather than using extra markup.

Also

<li>Create frames for your websites instantly with this cool 
<a style="font-family: verdana, arial, sanf-serif;font-size: 12px;text-decoration: underline; color: #000000;" href="http://www.freehtmlab.com/generators/framegen.html">frame generator.</a></li>

These inline styles would be better off taken out into the stylesheet rather than repeating them for each link.