Site Review

I got this page

www.the-hive.dk/~donp/

I would like if some would look at it and give back some constructive criticism

Firstly, your page is somewhat broken in Opera (v7.23). The menu category bars extend out of the menu region.

Second, try and introduce some padding, it is a wise idea, seperates the text from its borders.

Third, it is nice to see that you have used a tableless layout, however, your code does not validate @ HTML 4.01.

Fourth, you state font-size as a fixed size, in this case, .pt, it isn't wise, use %'s instead. That will allow people who use IE to increase the font size. Your page should be flexible enough to allow that.

Last, the layout/colour scheme. It seems a little packed into a small place. But that isn't very important, it is the colours/font sizes that you use that cause it to look cramped.

The colours do not compliment each other, and clash with the background colour that you have used. Try using a dark shade of brown instead of the navy blue. Further, there is no way of telling what is a link, well not really. Try and introduce hover links. It's more user friendly.

That is about all I can think of, anything else is minor, jibber jabber.


:)

It isn't very visually interesting. Try adding something to catch attention, and then let it go. ala: http://evolt.org/article/rating/22/60261/index.html

Also read up on some colour theory, because your colours just don't go together. See: http://www.worqx.com/color/index.htm

In both of the above links you will find the theories behind good visual design. It is up to you to actually use them :)

Later,
Douglas

Actually, it's best to use em's instead of %'s (btw, ionsurge, i noticed that was your 1234th post :))

As for the design, I would suggest going with a different (darker) blue for headings. It's to bright compared to the "mustard" yellow of the heading background. I also mught suggest changing the white borders to black or something darker, as they don't contrast enough.

You might also want to change the font to a sans-serif font like Verdana (Times just doesn't look right to me.)

-Andrew

Actually, it's best to use em's instead of %'s

Best to use both:

body { font-size: 80%; font-family: .... }
h1 { font-size: 1.2em; }
p { font-size: 1em; }

general font information...
http://thenoodleincident.com/tutorials/box_lesson/font/index.html
and concludes with...
http://www.thenoodleincident.com/tutorials/typography/index.html

But anyway...

Douglas

Layout is straightforward, there's not too much visual interest, but that's fine by me.
I would recommend having the header as seen on the home page return on the other pages as well to create some sort of unity.
Codingwise, everything looks pretty tidy; if you look into possible validation errors I'm confident that you can get it XHTML 1.0 strict. not necessary, but aim for excellence, right? :D
I would however like to point to the over-use of span elements; things like: <span class="Headline"> can easily be replaced by: <h1> and styled to taste; users with non-graphical browsers will like it, and so will search engines. Unless you don't want to be found, of course... :p

Best to use both

Using both, that is wrong. It simply eradicates the purpose of what I said, the browser will simply use the latter statement, e.g. if it was a fixed size, e.g. 14px, the browser would use that, rather than the previously stated % or vice versa.

You only need to use one statement, unless you are using different size values. Remember you can use values above 100%, e.g. 150% for larger text. :)

Changed the blue to brown.
made some padding.

gonna change to h1,2,3 tags as suggested they fill less bytes, I was under the impression though that a document where only allowed to have 1 of each.

as for not being able to distinguish links from the rest of the stuff I am a bit confused seing as all links are colored Teal.

and for not validating at the 'oh so migthy' w3c doen't really matter that much to me, the fault anyway is in how I write <ul> and <li> tags