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
Rosalee Johnson
5,726 PointsI am being asked to set the top and bottom margins to 0 and lefthand right margins to 10px. I have done so, no success.
Here is the short-hand version that I've provided:
nav ul {
margin: 0 10px; }
...and this is the long-hand version that I've provided:
nav ul {
margin-top: 0; margin-bottom: 0; margin-left: 10px; margin-right: 10px; }
... and I was still told to set the top margin 0. What am I doing wrong?
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;
}
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;
}
#nav ul {
margin-top: 0;
margin-bottom: 0;
margin-left: 10px;
margin-right: 10px;
}
2 Answers
Steven Parker
229,771 Points
Your selector is not targeting the right elements.
The challenge says: "Select the unordered list nested inside the nav element." But your selector is targeting unordered lists inside an element with the ID of "nav", since the pound sign (#) indicates an ID instead of an element type. Remove the "#" and you should pass the challenge.
And FYI, while the rest of your code is perfectly fine as is, you could accomplish the same thing more compactly with a single line:
margin: 0 10px;
Rosalee Johnson
5,726 PointsI appreciate the quick response! I actually tried the single line the very first time (within my examples in the initial message), as well. But I did remove the pound sign, as you suggested, and I was able to move to the next step. Thanks, Steven!