Posts: 1,270
Name: Abel Mohler
Location: Asheville, North Carolina USA
|
Let me guess. You're using Dreamweaver?
Not to change the subject, but shouldn't you be naming your CSS styles something more semantic, instead of .style1 .style2, etc? Also, don't put your styles in the head, unless they are being dynamically generated (which takes a lot of knowledge anyway). Place all of your CSS into an external file, and link to it using the <link> tag in the <head>.
This line (directly after your opening body tag), is horrible, just horrible:
HTML Code:
<div style="margin:0px auto;;width:900px;border:1px solid black;background:white;padding:0px;text-align:center;border-color:#666666;">
<div align="center">
<table width="900" height="650" border="0" cellpadding="0" cellspacing="0">
What you see above is a wrapper div with all inline styles wrapped around another div with a depreciated method for centering wrapped around at table that is being used for layout, and coincidentally, has the same width as the first div.
If I were you, I would start the unlearning process as soon as possible, before you become too good at doing things wrongly. This type of layout will get you nowhere, either professionally or in practice.
Last edited by wayfarer07 : 07-09-2008 at 11:17 AM.
|