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

Posted by Gidon Urs Bohnacker
Gidon Urs Bohnacker
677 Points

max-width and link between css and html

I put in my css max-width correctly but its still telling me I have to check it. Besides of that i linked my html code with my css but its not displaying it at all. Really Thankfull for help.

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

# wrapper {
  max-width: 940px;
  margin: 0 auto;
  }
index.html 
<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8">
    <title>Gidon Bohnacker | 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>

2 Answers

Amar Torres
Amar Torres
5,483 Points
Amar Torres
Amar Torres
5,483 Points

I think you may have your # wrapper wrong. There should be no space so the code should look something like 

#wrapper {
    max-width: 940px;
  }

I believe if you fix this it may take care of the width error as well as it not showing on the page.

Steven Parker
Steven Parker
231,248 Points
Steven Parker
Steven Parker
231,248 Points

It looks like you have a typo.

The "a" rule seems to be missing the closing brace. Also there seems to be a stray space between the "#" character and the ID name.

a {
  text-decoration: none;
}                    /* <- add closing brace */

#wrapper {           /* <- remove space between "#" and "wrapper" */
  max-width: 940px;
  margin: 0 auto;
}