08-09-2009, 12:43 AM
|
#7 (permalink)
|
|
The Addict
Join Date: May 2009
Posts: 287
Thanks: 5
|
- Flash is bad, it doesn't work for text readers/mobile browsers. (A mobile site would suffice for the later).
- The overall site load is pretty slow:
- gzip the files
- Use CSS sprites, compress images.
- Do [black, gray, white, lime, orange, [dark blue, light blue]] really go with each other? I wonder how it would look with out the green (orange and blue match very well).
- Is the comment form really a PM box that is linked with the forum? If so it is really nice.
- The text seems just a tad small and the font-face could be changed.
- Your Google Analytics code should really go on the bottom (just before the end of </body>, Page on why it will also speed up the page load.
- Tables? Maybe try using <div>'s?
- You seem to have a LOT of [IF IE x] conditional statements, this will slow page rendering. I would recommend JavaScript or PHP selective parsing. (Use JS or PHP to print/do specific things based on the browser, they are faster.)
- You're using HTML elements that are:
- Going to be deprecated in HTML5 (<font>).
- Can be replaced with <span>'s and CSS.
- Oh, you don't have a doctype.
- Always, Always fill in the alt="" attributes, they are used by screen readers or if your images go offline/[server, client] can't load them.
Overall: 6.5/10
You clearly have a grasp on what you've done, it seems that you are just a little behind in the current technologies (5,6,7[,8],9[,11]). The site is NOT bad, I just think it could be modified a little.
My $0.02.
- Adam
*EDIT* Formatting issues.
|
|
|
|