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![](https://ecs-static.teamtreehouse.com/assets/views/marketing/shared/community-banner-white-47072046c51352fe6a69f5e691ff5700b28bb11d45197d7bdf066d9ea3f72d0c.webp)
![Danielle Grondman](https://uploads.teamtreehouse.com/production/profile-photos/35389/micro_danielle.jpg)
Danielle Grondman
4,852 PointsTask 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
2 Answers
![Scott Evans](https://secure.gravatar.com/avatar/35227e03fe277c59bc3b0a7d8cf720c7?s=96&d=https%3A%2F%2Fecs-static.teamtreehouse.com%2Fassets%2Fcontent%2Fdefault_avatar-445fbbabfc8dc9188fb5967fe43322ee0c3e0dd1e10f378bf8343784af5a13eb.webp&r=pg)
Scott Evans
4,236 PointsI 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;
}
![bothxp](https://uploads.teamtreehouse.com/production/profile-photos/303042/micro_poley_2.jpg)
bothxp
16,510 PointsActually 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;
}
![bothxp](https://uploads.teamtreehouse.com/production/profile-photos/303042/micro_poley_2.jpg)
bothxp
16,510 Pointsor is that considered bad practice ?
Scott Evans
4,236 PointsScott Evans
4,236 PointsIs your question regarding the fourth stage of that challenge Danielle?