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

Daniel Paver
Daniel Paver
1,904 Points

Trying to put the top and bottom padding to 15px and left/right to 10px

Hello, i just wondered what i'm doing wrong here with my padding for the 'li'? Thanks :)

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 {
  font-weight: 800;
  display:inline-block;
  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

Kevin Lassar
Kevin Lassar
22,440 Points

Hi Daniel,

The challenge asks you to set the padding on your nav links to 15px / 10px and font weight to 800. You have your padding property set on your nav list items instead.

I've posted the correct code below for reference.

a {
  text-decoration: none;
}

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

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

h1, h2 {
  color: #fff;
}

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

/**  Your nav li only needs the display: inline-block property. Remove your padding and font weight property from this. **/
nav li {
  display: inline-block;
}

/**  Paste your padding and font weight property here on your nav links **/
nav a {
  color: #fff;
  font-weight: 800;
  padding: 15px 10px;
}

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;
}
Daniel Paver
Daniel Paver
1,904 Points

so this is an anchor?, i see