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 CSS: Cascading Style Sheets Style the Basic Elements

center the ID wrapper?

What am I doing wrong? I tried putting margin as "margin: 0 50%", but it keeps giving me an error.

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

}

#wrapper {
  max-width: 940;
  margin: auto auto;
}
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>

3 Answers

Konrad Pilch
Konrad Pilch
2,435 Points

I give you a hint:

Set the top and bottom to 0
And left and right to auto

The coding goas clockside.

So : Top, Right, Bottom, Left
Shortcuts : 10px 20px; - meaning, 10px top and bottom, and 20px right and left.

Got it thanks.

So, I could just write it as,

margin: 0px auto 0px auto; ??

Konrad Pilch
Konrad Pilch
2,435 Points

Well, what are you doing with 0 0 auto?

You need to put it as

 margin: 0 auto;

You dont need a value, 0 is zero, it could be anything 0% , 0em, 0px. 0 is nothing ;) so dont need value. as 0px is nothing .

You still need 0

Tim Knight
Tim Knight
28,888 Points

In addition to the margin auto that other posters have mentioned you'll want to make sure you specify a unit on the width of the container.

a {
  text-decoration: none;
}

#wrapper {
  max-width: 940px;
  margin: 0 auto;
}

Any time you're specifying a number for size that is not zero you should really specific it's unit. The only time this isn't true (from what I'm aware) is when you're dealing with line-height since line height is actual a multiple rather than a unit size.

Konrad Pilch
Konrad Pilch
2,435 Points

But you havent specific a unit for your 0 here, Instead ur just telling to speify 0px auto : p

It works without width, when i align my text center, my footer copyright or my full width image and want text, u dont need to specify width.

Tim Knight
Tim Knight
28,888 Points

Without a width your essentially centering something that is 100% wide. See http://codepen.io/timknight/pen/zvNpyz for example... and try removing the px on the width and you'll see what happens.

Konrad Pilch
Konrad Pilch
2,435 Points

If a unit is 0 , and u remove px, how much will it change? nothing .

And yes your right, we are talking about something else. Im talking about the margin and you about his width that didnt has px and i havent seen :) problem solved :)