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 Styling Web Pages and Navigation Create a Horizontal List of Links

Danielle Grondman
Danielle Grondman
4,852 Points

Task 4 doesn't work

My code is wright, 15px top, 10 px left and right padding on the nav links: nav li { padding: 15px 10px; }

It keeps on saying: Bummer, the top padding must be 15px!

Am I crazy

Scott Evans
Scott Evans
4,236 Points

Is your question regarding the fourth stage of that challenge Danielle?

2 Answers

Scott Evans
Scott Evans
4,236 Points

I ran through the Challenge myself and came up with this code that worked.

What i think you might have misread is the fourth stage asked you to select the <a> elements inside the <li> elements, so your selector should be nav ul li a { //css here }

For a full code example for that challenge see below, Hope this helps!

a {
  text-decoration: none;
}

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


nav ul { 
  margin: 0px 10px;
    padding: 0; 
  list-style: none;
}

nav ul li {
  display: inline-block;
}

nav ul li a {
  font-weight: 800;
  padding: 15px 10px;
}

#logo {
  text-align: center;
  margin: 0;
}

h1, h2 {
  color: #fff;
}

nav a {
  color: #fff;
}

nav a:hover {
  color: #32673f;
}

h1 {
  font-family: Changa One, sans-serif;
  font-size: 1.75em;
  font-weight: normal;
}

img {
  max-width: 100%;
}

#gallery {
  margin: 0;
  padding: 0;
  list-style: none;
}

#gallery li {
  float: left;
  width: 45%;
  margin: 2.5%;
  background-color: #f5f5f5;
  color: #bdc3c7;
}

Actually it says to select the links inside the nav element.

So using nav a { } will work fine.

nav ul {
 margin: 0 10px;
 padding: 0;
  list-style: none;
}

nav li {
 display: inline-block;
}

nav a {
  font-weight: 800;
 padding: 15px 10px;  
}

or is that considered bad practice ?