The forums ran from 2008-2020 and are now closed and viewable here as an archive.
Home › Forums › CSS › Class assignment critique?
check out the code for fonts and all. here’s a screenshot too:
Not sure if you know this but the font doesn’t work. Use fontsquirrel.
You should also have the previous slider hide when opening a new one. Also it could use some color.
1. You had seven missed closing divs (all of the sidebar stuff)
2. Your JS was violating a very important rule: DRY (Don’t Repeat Yourself)
Both of those fixed here: http://jsfiddle.net/mgmdp/2/
Another note: try to refrain from using ``. They shouldn’t be used for separating content, that’s what margins are for!