Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

HTML How to Make a Website Creating HTML Content Create Navigation with Lists

Posted by Tanner Mullen
Tanner Mullen
478 Points

glad i have evrything correct ut it is not working gj tree house

j

index.html 
<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8">
    <title>Nick Pettit</title>
  </head>
  <body>
    <header>
      <a href="index.html">
       <nav>
          <ul>
            <li><a href="index.html">Portfolio</a></li>
            <li><a href="about.html">About</a></li>
            <li><a href="contact.html">Contact</a></li>
        <h1>Nick Pettit</h1>
        <h2>Designer</h2>
      </a>
    </header>
    <section></section>
    <footer>
      <p>&copy; 2013 Nick Pettit.</p>
    </footer>
  </body>
</html>

3 Answers

Miguel de Luis Espinosa
Miguel de Luis Espinosa
41,279 Points
Miguel de Luis Espinosa
Miguel de Luis Espinosa
41,279 Points

Why are you linking the full <nav> element?

Joel Hernandez
seal-mask
.a{fill-rule:evenodd;}techdegree
Joel Hernandez
Front End Web Development Techdegree Student 2,571 Points
Joel Hernandez
.a{fill-rule:evenodd;}techdegree
Joel Hernandez
Front End Web Development Techdegree Student 2,571 Points

Hey there. You are forgetting to close a couple elements and some of them are in the wrong area. Your nav element and un-ordered list need to OUTSIDE of the anchor element for the H1 and H2. You currently have it nestled. You also need to close the NAV and UL tags. Here is what the header code should look like.

<header>
      <a href="index.html">
        <h1>Nick Pettit</h1>
        <h2>Designer</h2>
      </a>
      <nav>
        <ul>
          <li><a href="index.html">Portfolio</a></li>
          <li><a href="about.html">About</a></li>
          <li><a href="contact.html">Contact</a></li>
        </ul>
      </nav>
    </header>

Hope this helps!

Tanner Mullen
Tanner Mullen
478 Points

thank you

Shawn Boyd
Shawn Boyd
14,345 Points
Shawn Boyd
Shawn Boyd
14,345 Points

Try taking out the anchor that you open right below the header and close right after the h2. This is wrapping everything within your header to be a single link to index.html and messing up the links in your list items.