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

Jessica Pehrson
Jessica Pehrson
1,648 Points

whats wrong with this

what am i doing wrong?

css/main.css
a {
  text-decoration: none;
}
#wrapper {
  max-width: 940
  margin: center
}
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>

To center the #wrapper div you need to first specify a width, then set the margin property to auto. The keyword auto will automatically calculate the left and right margin of an element so that it is always centered.

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

2 Answers

 a{
  text-decoration: none;
}

#wrapper{
  max-width:940;
  margin:center;
}
Jessica Pehrson
Jessica Pehrson
1,648 Points

i tried those changes and it still says Im wrong. that i need to check my max-width

I forgot to mention that you were missing a semi-colon after max-width: 960px

Jessica Pehrson
Jessica Pehrson
1,648 Points

a { text-decoration: none; }

wrapper { max-width: 980px;

margin: auto; }

i just tried this and it still is wrong idk whats wrong about it

Sorry, I thought you were asked to center the div. I see that they want you to remove the underlines form all links. All you need to do is this

   a  {
  text-decoration: none
}

just seen the other step now. It looks you just have the wrong width value, it should be 940px. Try this.

a {
  text-decoration: none;
}

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