Hey congrats! :D Its lookin good too, well done :)
Here is my code critique :)
In ya body tag, remove all this leftmargin="0" topmargin="0" marginwidth="0" marginheight="0" and just add margin:0; padding:0; to ya totalBG class. Try using CSS Sprites instead of preloading images in the body tag. Here are some sprite tuts:
You don’t really need that <br />, just style the P correctly
All these "row" divs… again you might be away from tables, but you cant stop thinking like them eh? :D DIVs work how you want to – they don’t follow the same rules as tables :) you really need to break freeeeeeeeeeeeeeeeeeeeeee :D
All in all it was a great start! you have done well! Just need to move away completly from tables and stop trying to emulate them with divs :D
For a great PSD to XHTML/CSS tut – have a look here:
Well done! I like the site because it isn’t too garish. Some improvements would be to change the menu items so that their blur and hover effects are different. To be honest the glow effect is kinda old fashioned and cheesy so that’s why I would change it. Also going from light green to white for the background is a bad idea, I would stick to going from green to green – merely different shades.
I’m glad a first timer has come up with such a nice design. Keep up the good work!
Viewing 3 posts - 1 through 3 (of 3 total)
You must be logged in to reply to this topic.
*May or may not contain any actual "CSS" or "Tricks".