Please rate my layout!

[Ganon of Evil]

http://www.elite-gaming.com/finallayout

Any suggestions? Right now its designed for 1024x768 (im working on it for 800x600). Made it with 1stPage and PSP.

[ByteWizard]

Very clean. I am a big proponent of "un-jazzed" sites. Simple and straight to the point.

:thumbup:

[Ganon of Evil]

I understand what you mean by the navigation and have been working on it. I added a META tag to declare it as ISO-8859-1 so it should validate now (except for 6 errors which I am working on, its HTML 4.01 Transitional not XHTML). The layout is not completely done yet either (its called finallayout for no actual reason). Other forums have gave me tips to add a footer to it which I will be doing as well. Any other suggestions? And what exactly do you mean by a CSS layout, do you mean to use DIV tags to align everything?

Thanks for the suggestions,
Chris

[Zero Angel]

Well, for one thing, CSS layout means *not* using <font> tags at all.

<Font> is depreciated in HTML 4.0, meaning that there is a better alternative out there, namely CSS.


Using Non-breaking spaces to add padding is also a bad habit that bulks up your code and makes display unpredictable. Lets take a look at some of your code that uses archaic HTML and improve it so that it matches HTML 4.0 Strict Specs.

This is from your 'content' section of your page:<td height=35 valign=bottom align=right style="border-left: 1px solid #C4C4D0;"
width=626 background="images/top_content_bg.jpg">
<font color="#F8BF24" size=2>
<b>Subject goes here</b>&amp;nbsp;&amp;nbsp;&amp;nbsp;&amp;nbsp;</font>
</td>

This is the 'improved' version:<style type="text/css">
<!--
.contentheader { background: url('images/top_content_bg.jpg') middle center no-repeat;
text-align: right; vertical-align: bottom; border-left: solid 1px #c4c4d0;
width: 626px; height: 35px; font-size: 16px; font-weight: bold; color: f8bf24; padding-right: 20px; }
-->
</style>
....blah, blah, blah....
<td class="contentheader">
Subject goes here
</td>

As you can see, the code in the body portion of the document is simplified, and *all* of the style information is stored in the CSS portion of the document, which means that everything can be tweaked in one place, and if you change anything, it will be applied to all of the TDs with a class of 'contentheader'. Nifty, eh?

You can also further use CSS for layout instead of tables (which means a complex nesting of DIVs and SPANs), though it is a bit difficult to get the hang of, and has its own set of advantages and disadvantages. If you are using a template/include system, then I suggest that you stick with tables for now and work your way up to a full CSS layout when you feel comfortable.

[Ganon of Evil]

K i just never heard the term CSS layout. I know a lot of CSS but ill read a few more guides to get better (DIV tags are much easier to use for me then table tags, i just though tables were more compatible). Ill convert it to TABLES/CSS today (not DIV/CSS though).