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

Qasim Hafeez
Qasim Hafeez
12,731 Points

Padding issue in Code Challenge

As far as I can tell, I have set the top/bottom padding of the 'nav li' to 15px and the left/right padding to 10px. It keeps telling me that I need to set the top/bottom padding to 15px. What am I doing wrong?

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

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

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

h1, h2 {
  color: #fff;
}

nav a {
  color: #fff;
}

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

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

nav li{
  display:inline-block;
  font-weight:800;
  padding:15px 10px;
}

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;
}

2 Answers

Jennifer Nordell
seal-mask
STAFF
.a{fill-rule:evenodd;}techdegree
Jennifer Nordell
Treehouse Teacher

Yes, you set the padding to the li (list items) padding to 15px top and bottom and 10px to left and right. However, what it's looking for is the link (anchor tag) in the nav... not the list items. The resulting code you're looking for is as follows:

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

nav li {
  display: inline-block;
}

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

Hope this helps!

Qasim Hafeez
Qasim Hafeez
12,731 Points

Thank you so much! Who knows how long it would have taken for me to notice that1

Arman Arutyunov
Arman Arutyunov
21,900 Points

The thing is that they ask you in the challenge to select LINKS inside the nav.

nav ul {
  margin: 0 10px;
  text-decoration: none;
  padding: 0;
}

nav ul li {
  display: inline-block;
}

nav ul li a {
  font-weight: 800;
  padding: 15px 10px;
}
Arman Arutyunov
Arman Arutyunov
21,900 Points

Sorry, didn't see you've already got an answer