Forums

Give help. Get help.

  • # May 21, 2013 at 3:12 pm

    http://jsfiddle.net/attilahajzer/mgmdp/1/embedded/result/

    check out the code for fonts and all. here’s a screenshot too:

    https://twitpic.com/cs8qja/full

    Anonymous
    # May 21, 2013 at 3:48 pm

    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.

    # May 21, 2013 at 5:37 pm

    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!

Viewing 3 posts - 1 through 3 (of 3 total)

You must be logged in to reply to this topic.

icon-anchoricon-closeicon-emailicon-linkicon-logo-staricon-menuicon-nav-guideicon-searchicon-staricon-tag