site review

[JavaHead Jonnie]

could some people review my site for me? (Site: http://fightscene.50free.org Forums: http://fsforums.50free.org )

thanks

oh and i know that the urls don't match the name cos we're changing the name and style of a game (an MMORPG) that the site's for.

[DaveSW]

Where you have:
<iframe height="100%" width="100%" frameborder="0" src="/m_sign.html">Sorry, this site requires Iframes!</iframe>

Put:
<iframe height="100%" width="100%" frameborder="0" src="/m_sign.html"><a href="/m_sign.html">This page requires iframes, but click here to go to the page</a></iframe>

This makes it more accessible to very old browsers.

Codewise it looks ok, though I haven't run it through a validator. It looks ok in IE and firefox, so quite a good job. You might want to work a bit more on your graphics, but I'm not sure exactly what needs to be done to them. Maybe just reducing the filesizes would help!

[Paul Jr]

Validation Results (http://validator.w3.org:8001/check?verbose=1&uri=http%3A//fightscene.50free.org/)

[sharkey]

Codewise it looks ok

I cant find a doc type on it. :)

[DaveSW]

I was referring to the use of divs for layout. Most of the errors are a lack of alt tags, or in the unusual characters in the link code near the bottom of the page.

[JavaHead Jonnie]

Originally posted by DaveSW
[B]Where you have:
<iframe height="100%" width="100%" frameborder="0" src="/m_sign.html">Sorry, this site requires Iframes!</iframe>

<snip>

i know, i was just using iframes to test the page on my pc. now it's online, i'll be using <div style="overflow: auto;" ... > with php includes.

as for doctype - added.