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

CSS CSS Basics (2014) The Box Model Padding, Borders, and Margins

Suxin Deng
Suxin Deng
804 Points
Posted by Suxin Deng
Suxin Deng
804 Points

question about the third step question

Hi, I have a question concerning this one. " Target the .secondary-content div. Set the top padding to 80 pixels and the bottom padding to 70 pixels." I write the code as this but it shows I was wrong and "Make sure you're setting the correct top padding value inside the '.secondary-content' rule."

My code: div .secondary-content{ padding-top: 80px; padding-bottom: 70px; }

Can you help me with where did I go wrong? Thank you

style.css 
/* Complete the challenge by writing CSS below */
index.html 
<!DOCTYPE html>
<html>
  <head>
    <title>Lake Tahoe</title>
    <link rel="stylesheet" href="page.css">
    <link rel="stylesheet" href="style.css">
  </head>
  <body> 
        <div class="primary-content t-border">
            <p class="intro">
                Lake Tahoe is one of the most breathtaking attractions located in California. It's home to a number of ski resorts, summer outdoor recreation, and tourist attractions. Snow and skiing are a significant part of the area's reputation.
            </p>
            <a href="#more">Find out more</a>

      <div class="wildlife">
        <h2>Check out all the Wildlife</h2>
        <p>
          As spawning season approaches, the fish acquire a humpback and protuberant jaw. After spawning, they die and their carcasses provide a feast for gatherings of <a href="#mink">mink</a>, <a href="#bears">bears</a>, and <a href="#eagles">bald eagles</a>.
        </p>
      </div>

            <a href="#wildlife">See the Wildlife</a>
        </div><!-- End .primary-content -->

        <div class="secondary-content t-border"> 
      <div>
        <h3>From Tents to Resorts</h3>
        <p>
          Lake Tahoe is full of wonderful places to stay. You have the ability to sleep in the outdoors in a tent, or relax like a king at a five star resort. Here are our top three resorts:
        </p>
        <ul>
          <li><a href="#hotels">Lake Tahoe Resort Hotel</a></li>
          <li><a href="#resorts">South Lake Tahoe Resorts</a></li>
          <li><a href="#lodging">Tahoe Ski Resort Lodging</a></li>
        </ul>       
      </div>
      <div>
        <h3>Pack Accordingly</h3>
        <p>
          One of most important things when it comes to traveling through the great outdoors is packing accordingly. Here are a few tips:
        </p>
        <ol>
          <li>Bring layers of clothing</li>
          <li>Pack sunscreen</li>
          <li>Carry extra water just in case</li>
          <li>Pack light</li>
        </ol>
      </div>
        </div><!-- End .secondary-content -->

        <footer class="main-footer">
            <p>All rights reserved to the state of <a href="#">California</a>.</p>
            <a href="#top">Back to top &raquo;</a>
        </footer>
  </body>
</html>

5 Answers

Antria Sofroniou
Antria Sofroniou
2,803 Points
Antria Sofroniou
Antria Sofroniou
2,803 Points

.secondary-content{ padding:80px 0px 70px 0px; }

and also if you want, you don't need the div, and also you don't need to write it separately for top and bottom.

David Bath
David Bath
25,940 Points
David Bath
David Bath
25,940 Points

Remove the div selector before .secondary-content in your CSS code and it will be fine. As you have it now it is trying to target .secondary-content inside a div element.

Mark Truitt
Mark Truitt
17,230 Points
Mark Truitt
Mark Truitt
17,230 Points

Or you can remove the space and target any div with that class.

div.secondary-content.

So say I had multiple secondary-content. This would allow you to use it as a naming convention header footer etc etc. And target it by the container.

connor ingold
connor ingold
5,382 Points
connor ingold
connor ingold
5,382 Points

I figured it out it should be:

.secondary-content{ padding: 80px 0px 70px 0px}

Instead of:

.secondary-content{ padding-top: 80px 0px 70px 0px}

You need to remove where it says "top" as you are already specifying to make the padding top by putting the 80px first. Hope that made sense

Larissa Ragland
Larissa Ragland
6,653 Points
Larissa Ragland
Larissa Ragland
6,653 Points

.primary-content { padding-top: 25px; padding-bottom: 95px; } .secondary-content{ padding:80px 0px 70px 0px; }

.box { padding-top: 80px; padding-bottom: 70px; }

.wildlife { border-top-width: 10px; border-top-style: solid; border-top-color : orange; }