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 Center the Wrapper

followed all the steps on the video....div still not changing, please help

Followed all the steps so far, but still not seeing any changes when I save and refresh, looked at other responses still no help.

p.s. love this course so far :-)

this is what I have for html:

</header> <div id="wrapper"> <section> <ul> <li> <a href="img/numbers-01.jp"> <img src="img/numbers-01.jpg" alt=""> <p>Experimentation with color and texture</p> </a> </li> <li> <a href="img/numbers-02.jp"> <img src="img/numbers-02.jpg" alt=""> <p>Playing with blending modes in PS</p> </a> </li> <li> <a href="img/numbers-06.jp"> <img src="img/numbers-06.jpg" alt=""> <p>Trying to create an 80s style of glows.</p> </a> </li> <li> <a href="img/numbers-09.jp"> <img src="img/numbers-09.jpg" alt=""> <p>drips created using ps rushes</p> </a> </li> <li> <a href="img/numbers-12.jp"> <img src="img/numbers-12.jpg" alt=""> <p>creating shapes using repition</p> </a> </li> </ul>

  </section>
  <footer>
        <a href="https://twitter.com/bigmommaprods"><img src="img/twitter-wrap.png" alt="Twitter Logo"></a>
         <a href="https://www.facebook.com/Treehouse"><img src="img/facebook-wrap.png" alt="Facebook Logo"></a>

    <p>&copy; 2013 Nick Pettit.</p>
  </footer>
</div>

</body>

this is what I have for css:

a { text-decoration: none; }

wrapper {

max-width: 940px; margin: 0 auto; background: orange; }

2 Answers

Gunhoo Yoon
Gunhoo Yoon
5,027 Points

UPDATE: Thanks to Cody Te Awa I'm pretty sure you actually put '#' symbol.

So here's more thing that you could've done wrong. Actually, I can only think of one.

1 You forgot to save your progress and did not refresh the preview page.

If you've done above and still not getting result try to change the selector to div instead of #wrapper. If that still doesn't work comment me.

You are trying to select <div id="wrapper"> but you missed the pound symbol '#' which refers to id of element.

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

Alternatively, for this specific case you can also do

div {
    max-width: 940px; 
    margin: 0 auto; 
    background: orange; 
}
Cody Te Awa
Cody Te Awa
8,820 Points

just pondering, I think they may have used the pound symbol because I think that is the symbol which makes the word stand out like it is, for eg

this sentence has # at the start

Gunhoo Yoon
Gunhoo Yoon
5,027 Points

Thanks for correcting me that. I'll double check.

Update: I just checked and it worked

finally got it to work thank you