Click to See Complete Forum and Search --> : site review


krazykris
01-18-2004, 12:01 PM
Hi guys,

I have just recently finished my new layout and i'm also giving the coding ago my self, however i've been informed that it does not display correctly in the Mozilla browser, it displays correctly in Internet Explorer though.

Plz can you give me some tips on how to correct this, also constructive critism is always welcome.

ps-heaven (http://www.ps-heaven.co.uk)

Thanks

krazykris
01-18-2004, 12:18 PM
:( ...I'm guessing that no-one can help me out

Vladdy
01-18-2004, 12:44 PM
Way too slow.
Layout techniques of the last century result in a bloated code.
Top menu is text represented by images - any good reason for that???
Navigation and content can use some padding.
Pootr contrast hard to read.
Your background images are misaligned (in Firebird)

ray326
01-18-2004, 02:46 PM
Wouldn't the easiest thing to do be to download Mozilla or Firebird and look at it yourself? If something's not right, I sure don't see it in my Moz 1.6.

As far as feedback goes, it looks ok to me but the page would be a lot smaller if you realized it using CSS rather than your table layout.

spufi
01-19-2004, 01:42 AM
Don't have the "Latest Forum" info scroll. A good rule in general is, scrolling = bad. I would also change the color of the links also in the "Latest Forum" section. The default blue that links have kind of sucks, IMHO.

iniquity101
01-19-2004, 10:53 AM
i would personally have the "contact me" or "webmaster" etc. as the link text not the >>. This is only because i like a big link, not something i have to find with my mouse:D

I also think the flash banner on "where can you find..." scrolls through News Fortums etc to fast.

krazykris
01-19-2004, 07:26 PM
thanks for the feedback guys,

I'm gunna get on and make those changes you mentioned, it now works in all browsers :D

Paul Jr
01-19-2004, 07:33 PM
Also, in your external stylesheet, remove this from the top:
<style type="text/css">
<!--
And this from the bottom:
-->
</style>

krazykris
01-20-2004, 05:20 PM
ok i will, but why do i have to remove it?

Paul Jr
01-20-2004, 07:43 PM
Because that's an external stylesheet, which is only meant to contain CSS Styles, and not HTML Tags.

krazykris
01-20-2004, 07:47 PM
oh right...thanks