treehouse : what would you like to learn today?
Web Design Web Development iOS Development

Please Analyse my HTML Code And Give Feedback

  • So how does my code stand? are there any unusual or unrecommended methods or errors? please let me know.http://codepen.io/Jarolin/pen/aEyjb

    PS. It looks better than that. Just couldn't link some images.

  • This should get you going in the right direction (note that I've quickly coded CSS just to get a feel for the page)

    http://codepen.io/anon/pen/xIAgL

    In your original code, you have numerous syntax errors. Run your code trough HTML validator. Also, you can not use same ID more than once on the page. Ex. id="fnav"

  • Agree with @AlenAbdula, you can't use same ID more than once.

    id="fnav"
    
  • huh. I actually got rid of that a few days ago and is not on the original HTML code. But thanks anyway.

  • Your input elements are badly formed

           <input id="daily" type="radio">Daily</input>
    

    should be

           <input id="daily" type="radio" value="Daily">
    

    They also don't have labels.

  • I actually got rid of that a few days ago and is not on the original HTML code

    Are you going to provide a new Codepen...or can this be marked [Solved]?

  • A few notes:
    * As Paulie said, you need to connect a label to each input. This is accessibility 101.
    * You really should use lists when you are dealing with lists. It is the point.
    * Don't use <br> when you can use padding/margin. It is not the point of a <br> tag.
    * You should use reusable class names. "one", "two", "three" are not great but I'm nitpicking.
    * Please add alt="" tags to your images. Once again, accessibility 101.
    * There are a couple of missing attributes (form misses action=""; button misses type="submit").
    * I don't want to feed the troll but you should use classes instead of ID.
    * As far as I know, you shouldn't use nav for the footer navigation. nav aims at targeting the main navigation.