View Single Post
Old 01-09-2013, 12:22 PM   PM User | #2
AndrewGSW
Senior Coder

 
Join Date: Apr 2011
Location: London, England
Posts: 2,120
Thanks: 15
Thanked 354 Times in 353 Posts
AndrewGSW will become famous soon enough
Best practices:

Don't use tables for layout. If you search CSS navigation bar there are many tutorials to study.

Code:
<td width="100"> <font color="white">&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp</font></td>
Height and width attributes for td and tables, etc., are deprecated: use CSS.

The font tag is obsolete.

Don't use nbsp's repeatedly, this is a poor practice; use margins to separate elements (or float elements, etc.).

Code:
#top {
	margin-left: 0 auto; /* margin-left/right only have one value */
	margin-right: 0 auto; /* you could write both as margin: auto 0; 
 - though it's normally margin: 0 auto; to center an element */
	position: absolute; /* absolute positioning should be a last resort */
	z-index: 1;
}

#under {
	margin-left: 0 auto; /* notes as above */
	margin-right: 0 auto;
	position: absolute;
	z-index: 2;
}

#container {
	clear: right; /* there is nothing floated to clear */
	display: inline;
	padding: 0px 0px 0px 0px; /* padding: 0; */
	margin : 0;
	margin-left:auto;
	margin-right:auto;
	float: center; /* there is no center value */
}
I think you need to spend some time studying CSS.
__________________
"I'm here to save your life. But if I'm going to do that, I'll need total uninanonynymity." Me Myself & Irene.
Validate your HTML and CSS
AndrewGSW is offline   Reply With Quote