OZONE Asylum
Forums
Site reviews!
New site layout
This page's ID:
10441
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
Much better than the last one :) Code wise there isn't much to comment on. The only thing I noticed is that you're using <h1> tags for the 'member spotlight' and 'gift cert...' titles. It is good that you're using a <h> tag for it, but the <h1> tag should usually be used only for page titles, in your case the top 'Curves' text. It isn't a big deal, but you should switch those from <h1> to <h2>. If you want to get really picky about it you should display the top graphic ('Curves'-image) using CSS to better your position in search engines. How you do this is explained in [url=http://www.digital-web.com/features/feature_2003-08.shtml]this article from Digital Web[/url]. Your code also lacks a small little thing to validate as XHTML 1.0. The <img> tag on line 11 isn't closed and it lacks an alt attribute. It should look like <img href="..." alt="some fancy text" />. Another nitty picky thing about your code is that your navigation should be a list since it in reality is a list of links. How to do this is explained over at [url=http://www.alistapart.com/stories/taminglists]A List Apart[/url]. There is another little thing about the menu. The link leading to the page that the user is currently on should not be a link, since it won't really take the user anywhere. That link should also be displayed differently than the other ones so the user easily can see where he is on your page. In your CSS file you've also used px to specify the font size. This is generally a bad idea. I every browser (except IE) the user can change the font size even if you use absolute measurements, so you might as well use relative sizes from the beginning and that way giving IE users the same option. You've also specified the widht of the page in px. I don't see any reason why you can't use % to specify the width. That way it would look good on more resolutions, since it will look rather small on a large resolution as it is now. I'm not that much of a fan of the colors you use, but that is probably because I don't like that pinkish tone. So considering that I don't like the color from the beginning I think you've done a very good job with it. But the link color could need some adjustments. Right now it is rather hard to read against the white background. Maybe darken it a tad so it contrasts better with the background. The contact link should lead to a contact page, not directly to a mail adress. The reason for this is mainly that the link looks exactly like the others so the user would expect the link to behave similarily. It might also seem 'more proffesional' if you gave the user a contact form to use along with an optional e-mail adress. Giving away the adress in clear text like you've done a couple of places are also a certain way to make sure you recieve spam mail. It would be better if you used some fancy document.write javasript to display the mail. This way the user would see the mail as normal, but the spam bot would only read the javascript which it wouldn't exepct as a mail. Keep up the good work! _________________________ "There are 10 kinds of people; those who know binary, those who don't and those who start counting at zero" [This message has been edited by Veneficuz (edited 09-16-2003).]
Loading...
Options:
Enable Slimies
Enable Linkwords
« Backwards
—
Onwards »