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 Responsive Web Design and Testing Refactor the Layout

Posted by James Rennie
James Rennie
16,807 Points

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,271 Points
james south
.a{fill-rule:evenodd;}techdegree seal-36
james south
Front End Web Development Techdegree Graduate 33,271 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.

James Rennie
James Rennie
16,807 Points
James Rennie
James Rennie
16,807 Points

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