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

Matt Young
Matt Young
180 Points
Posted by Matt Young
Matt Young
180 Points

Link each of the three list items...

I'm trying to get this accepted, but it says this portion of the code is no good. I've reviewed it and can't find the error.

<li><a href="index.html">Portfolio</a></li>

Isn't that linking portfolio to index.html?

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

2 Answers

Abe Layee
Abe Layee
8,378 Points
Abe Layee
Abe Layee
8,378 Points

Your code is almost correct except you added the a,h1, and h3 tag in the wrong place. The ul is always follow by li tag. You shouldn't have the h1 and h2 tag inside ul element.

<a href="index.html">
        <nav>
          <ul>            
           <h1>Nick Pettit</h1><!--this shouldn't be here-->
           <h2>Designer</h2><!--your mistake here too-->
             <li><a href="index.html">Portfolio</a></li>
             <li><a href="about.html">About</a></li>
             <li><a href="contact.html">Contact</a></li>
      </a><!--this shouldn't be here-->
        </ul>

Correct way

        <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>
Matt Young
Matt Young
180 Points
Matt Young
Matt Young
180 Points

That did it. Thanks a lot.