OZONE Asylum
Forums
Site reviews!
Site Check Please
This page's ID:
10525
Search
QuickChanges
Forums
FAQ
Archives
Register
Edit Post
Who can edit a post?
The poster and administrators may edit a post. The poster can only edit it for a short while after the initial post.
Your User Name:
Your Password:
Login Options:
Remember Me On This Computer
Your Text:
Insert Slimies »
Insert UBB Code »
Close
Last Tag
|
All Tags
UBB Help
Looks good, very good work to be your first all css page. Tested in both Opera7.5 and Firefox on Linux. First thing that struck me as not so good was the link (background-)color. It doesn't really fit in with the rest of the design and creates to big contrast with the rest of the text. I think I would use a simple underlined link style with a yellowish color instead, and then make the color more yellow when it is hovered over. If not I would at least add some more padding to the current link style, the background color is to thight to the text almost making it hard to read. I also find the text a little bit hard to read. I replace the background behind "maincontent" with an identical, but darker background. The css style for creat ing something something like that would be: [code] body { background: #aaa url(image.jpg) 0 0 no-repeat fixed .... } #maincontent { background: #aaa url(image-darker.jpg) 0 0 no-repeat fixed .... } [/code] -[url=http://www.meyerweb.com/eric/css/edge/complexspiral/demo.html]Example at css/edge[/url]- That way the background behind the text would be darker, and the contrast with the text would be bigger and therefore make the text easier to read. The menu could also need some more work. I like the style of it, but it seems lacking compared to how good the rest of the page looks. First off I would add a greater margin between the elements and some more padding on each link. It feels kind of cramped now. Next is the hover effect that doesn't really fit in. I think it would look better if you only underlined and changed the text color on hover. Have you tried having the menu left-aligned in the white box, instead of centered? Not sure, but that might look better since it feels out of sync where it is now. It isn't aligned to any of the browser borders, but it isn't aligned to any other major elements on the page either. So having it left aligned might work better. And remove the "best viewed 1024x768", the page looks good at 1280x1024 as well. Code wise there are a few things that could be improved. You should have the menu as a list. A list is the sematicly correct way of coding a menu and it makes for cleaner code as well. For ideas on how to manipulate lists to look how you want check out[url=http://css.maxdesign.com.au/listamatic/]Listamatic[/url]. You've got a couple of empty <p></p> tags in your code. Those should be removed. I'm not absolutely sure about this, but I think that you should only have one <h1> element on your page. At the moment you've got two, change the second one to <h2> and you'll be fine. I would also remove the scrollbar styles, but there is not real harm in having them there. But anyway you should convert them to lower case. To have the page validate you need to add 'type="text/css"' to the style tags surrounding the scrollbar styles. You should also 'upgrade' your doctype to XHTML1.0 or XHTML1.1, I think your page would validate as that as well. _________________________ "There are 10 kinds of people; those who know binary, those who don't and those who start counting at zero" - [url=http://www.golden-ratio.net]the Golden Ratio[/url] -
Loading...
Options:
Enable Slimies
Enable Linkwords
« Backwards
—
Onwards »