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 How to Make a Website CSS: Cascading Style Sheets Style the Basic Elements

Ryan Tana
Ryan Tana
303 Points
Posted by Ryan Tana
Ryan Tana
303 Points

What value do I put in to center my margin property?

I'm on a challenge task and it is asking me to "Select the ID wrapper, set it to a max width of 940 pixels, and then center it on the page using the margin property." I'm pretty sure I have the first two steps done correctly, but it is telling me to "Check the margin property." Please help?

css/main.css 
a {
  text-decoration: none;
}

#wrapper {
  max-width: 940 px;
  margin:
}
index.html 
<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8">
    <title>Nick Pettit | Designer</title>
    <link rel="stylesheet" href="css/normalize.css">
    <link rel="stylesheet" href="css/main.css">
  </head>
  <body>
    <header>
      <a href="index.html" id="logo">
        <h1>Nick Pettit</h1>
        <h2>Designer</h2>
      </a>
      <nav>
        <ul>
          <li><a href="index.html" class="selected">Portfolio</a></li>
          <li><a href="about.html">About</a></li>
          <li><a href="contact.html">Contact</a></li>
        </ul>
      </nav>
    </header>
    <div id="wrapper">
      <section>
        <ul>
          <li>
            <a href="img/numbers-01.jpg">
              <img src="img/numbers-01.jpg" alt="">
              <p>Experimentation with color and texture.</p>
            </a>
          </li>
          <li>
            <a href="img/numbers-02.jpg">
              <img src="img/numbers-02.jpg" alt="">
              <p>Playing with blending modes in Photoshop.</p>
            </a>
          </li>
          <li>
            <a href="img/numbers-06.jpg">
              <img src="img/numbers-06.jpg" alt="">
              <p>Trying to create an 80's style of glows.</p>
            </a>
          </li>
          <li>
            <a href="img/numbers-09.jpg">
              <img src="img/numbers-09.jpg" alt="">
              <p>Drips created using Photoshop brushes.</p>
            </a>
          </li>
          <li>
            <a href="img/numbers-12.jpg">
              <img src="img/numbers-12.jpg" alt="">
              <p>Creating shapes using repetition.</p>
            </a>
          </li>
        </ul>
      </section>
      <footer>
        <a href="http://twitter.com/nickrp"><img src="img/twitter-wrap.png" alt="Twitter Logo"></a>
        <a href="http://facebook.com/nickpettit"><img src="img/facebook-wrap.png" alt="Facebook Logo"></a>
        <p>&copy; 2014 Nick Pettit.</p>
      </footer>
    </div>
  </body>
</html>

1 Answer

Brian Jones
Brian Jones
5,803 Points
Brian Jones
Brian Jones
5,803 Points

If you set margin to auto then the element will be centered if it has a specified width.

Ryan Tana
Ryan Tana
303 Points
Ryan Tana
Ryan Tana
303 Points

I think that solved the margin property issue, but now it's telling me to check the max-width value. This is what I have now:

wrapper {

max-width: 940 px; margin: auto; }

Brian Jones
Brian Jones
5,803 Points
Brian Jones
Brian Jones
5,803 Points

Sorry, missed that. You need max-width: 940px (no space between 940 and px)

Ryan Tana
Ryan Tana
303 Points
Ryan Tana
Ryan Tana
303 Points

solved it. it was the space between 940 and px. thanks for the help!