Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

HTML How to Make a Website Responsive Web Design and Testing Refactor the Layout

Why does bummer say something different then the code challenge?

Bummer! Be sure you're only setting the width on every 4th line?

If I set the width only on every 4th line it says task 1 is wrong. If I leave the width changing on every line from task one it says task 2 is wrong. Confused how to fix this is this a bug?

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

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 {
  list-style: none;
  margin: 0 10px;
  padding: 0;
}

nav li {
  display: inline-block;
}

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

.profile-photo {
  display: block;
  margin: 0 auto 30px;
      max-width: 150px;
  border-radius: 100%;
}

.contact-info {
  list-style: none;
  padding: 0;
  margin: 0;
  font-size: 0.9em;
}

.contact-info a {
  display: block;
  min-height: 20px;
  background-repeat: no-repeat;
  background-size: 20px 20px;
  padding: 0 0 0 30px;
  margin: 0 0 10px;
}

@media screen and (min-width: 480px) {
  #gallery li {
 width: 28.3333%;
  }

 #gallery li:nth-child(3) {
   clear: left;
  }


}

2 Answers

james south
seal-mask
.a{fill-rule:evenodd;}techdegree seal-36
james south
Front End Web Development Techdegree Graduate 33,258 Points

nth-child(3) only targets the third child, not every 3rd child. for that you need 3n. when it says a previous task is no longer passing it usually means you have an error in the current task. 4n would be every 4th, 4 would be the 4th only. 3n + 2 would be every 3rd, starting at the 2nd, etc.

Interesting thank you for your explanation now it works. I got confused and thought:

gallery li:nth-child(3) {

clear: left; }

was correct as when I entered that the error message changed. I would have thought:

gallery li:nth-child(4) {

clear: left; }

would have changed the error message rather then having it set as 3.

Also my interpertation of:

@media screen and (min-width: 480px) { #gallery li { width: 28.3333%; }

#gallery li:nth-child(4n) { clear: left; }

}

would be that

gallery li {

width: 28.3333%; }

gives width 28.3333% to all gallery list items and that:

gallery li:nth-child(4n) {

clear: left; }

clears float on left side on every 4th list item?

So why does the error say Bummer! Be sure you're only setting the width on every 4th line?

Am I not setting the width on every line? Hopefully that all make sense and someone can explain why?

Thanks,

James