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 HTML Basics Structuring Your Content Structuring Content Challenge

What am I doing wrong?

What am I doing wrong?

index.html
<!DOCTYPE html>
<html>
  <head>
    <link href="styles.css" rel="stylesheet">
    <title>My Portfolio</title>
  </head>
  <body>
    <nav>
      <ul>
        <li><a href="#">About</a></li>
        <li><a href="#">Work</a></li>
        <li><a href="#">Contact</a></li>            
    </ul>
    </nav>
    <section>
      <h1>My Web Design &amp; Development Portfolio!</h1> 
      <p>A site featuring my latest work.</p>
    </section>
    <h2>Welcome</h2> 
    <p>Fusce semper id ipsum sed scelerisque. Etiam nec elementum massa. Pellentesque tristique ex ac ipsum hendrerit, eget feugiat ante faucibus.</p>
    <ul>
      <li><a href="#">Recent project #1</a></li>
      <li><a href="#">Recent project #2</a></li>
      <li><a href="#">Recent project #3</a></li>     
    </ul>

    <p>&copy; 2017 My Portfolio</p>
    <p>Follow me on <a href="#">Twitter</a>, <a href="#">Instagram</a> and <a href="#">Dribbble</a></p>
  </body>
</html>

1 Answer

Shay Paustovsky
Shay Paustovsky
969 Points

Hi Alex, There are a few mistakes in your code but you're on the right way.

TASK-1 : place the <ul>,<h1>,<p> inside an element that represent introductory content.

<header>
   <ul>
      <li><a href="#">About</a></li>
      <li><a href="#">Work</a></li>
      <li><a href="#">Contact</a></li>            
    </ul>
    <h1>My Web Design &amp; Development Portfolio!</h1> 
    <p>A site featuring my latest work.</p>
</header>

the <header> element represents a group/section of introductory content

TASK-2 : You're asked to put the <p> tags inside an element that usually contains information of the side, copyrights, social media links and more at the bottom of the site.

<footer>
      <p>&copy; 2017 My Portfolio</p>
      <p>Follow me on <a href="#">Twitter</a>, <a href="#">Instagram</a> and <a href="#">Dribbble</a></p>
</footer>

TASK-3 : Everything is excellent, the <ul> is properly indented inside the <nav> element which btw represents a major navigation section on your page.

P.S Usually the body tag divides itself into 3 major section: <header>, <main>, <footer> their purpose I've explained above. the header most commonly includes a introductory content (<p> + heading) with a navigation menu <nav>. The <section> element which you've used to wrap the <p> & <h1> tags is incorrect because the <section> tag represents thematic breaks between sections of the page.

HAPPY CODING

Thanks a lot:)