Review My Website — OZONE Asylum, home of the Mad Scientists

Review My Website

This page's ID: 22301

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	There are a couple of next steps that should occur. First you should now pull your style sheet out of the main page and make is external. <style type="text/css" media="screen">@import "/s/style.css";</style> The other thing I don't like is your classitis. You are using both classes and inline styles way to much when they are not needed, and by doing this you are providing some additional "stucture" in your document that is not needed. The huge perpetrator in this is your calendar. You have repeated class="cal" some 30 times. What you should do instead is to give your calendar table an id attribute of calendar. You can then manipulate the td elements via #calendar td and you don't have to worry about the class. If you need to have some special style on some of the random elements, then specify a class for them. Specifying inline the color of the calendar elements is terrible. Using the calendar id set on your table you could then do the following #calendar td { color: #666 } #calendar a { color: blue; text-decoration:none } #calendar a:hover { text-decoration:underline } That there would get rid of a whole lot of extra code. This type of extra code is what CSS aims to remove. By putting it in your are defeating the reason for using CSS, which is to make your life easier. By having all of these inline styles it makes your style switcher a real pain in the ass to work with. In the end to use the style switcher you should only have to change the style sheet and the look and feel of the entire website should be able to completely change. Some more issues. You should change all <b> elements to <strong>, you should change all <i> elements to <em> these are the new elements that replace the meaningless old ones. You should also look into using headers. There is no reason you should have anything resembling <font size="5"><b>News/Blog</b></font> This should be replaced with <h1>News/Blog</h1> and then manipulate the style sheet for that particular <h1> attribute (based on what it is a sub element of). You should have a number of levels of header tags. Using this attribute gives you page lot more powerful structure. Try using the "outline" feature of the HTML validator. If you want to know what the outline should look like you might try getting it for my site http://www.codetown.org, this just gives you a little example. The real reason for doing this other than it makes things easier to code, is that search engines put more importance on elements in the header tags. So if you are trying to get indexed as a "news/blog" site type thing, google will play up the "news/blog" angle because it is in a header attribute. Finally, I don't know how the heck this site is validating. You should at least be running into errors due to the use of the "font" tag. This elements has been depricated, which means that you should not be using it anymore. I know I put down a whole big number of issues here. Don't think you are doing everything wrong. The oposite is true, you are making some great headway, when you are this close to having something really good, that is the time when you start getting hammered. Dan @ [url=http://www.codetown.org]Code Town[/url] Loading...
Options:	Enable Slimies Enable Linkwords

« Backwards — Onwards »

Show Forum Drop Down Menu