Thread: Heratheim Works
View Single Post
Old 08-09-2009, 12:43 AM   #7 (permalink)
adamdecaf
The Addict
 
Join Date: May 2009
Posts: 287
Thanks: 5
adamdecaf is on a distinguished road
Default

  1. Flash is bad, it doesn't work for text readers/mobile browsers. (A mobile site would suffice for the later).
  2. The overall site load is pretty slow:
    1. gzip the files
    2. Use CSS sprites, compress images.
  3. 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).
  4. Is the comment form really a PM box that is linked with the forum? If so it is really nice.
  5. The text seems just a tad small and the font-face could be changed.
  6. 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.
  7. Tables? Maybe try using <div>'s?
  8. 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.)
  9. You're using HTML elements that are:
    1. Going to be deprecated in HTML5 (<font>).
    2. Can be replaced with <span>'s and CSS.
  10. Oh, you don't have a doctype.
  11. 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.
__________________
My Site
adamdecaf is offline  
Reply With Quote