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 Make a CSS Image Gallery

Deepak Chaudhary
Deepak Chaudhary
1,369 Points

I'm stuck here!

It says to fill the parent element with images and did all that margin and max-width stuff but it is not working. :(

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;
}
ul{
  list-style:none;
}
#gallery{
  max-width:100%;
}

#gallery img {
  max-width:45%;
  margin:2.5%;
  padding:0;
  float:left;

}

#gallery{
  background:grey;
}
body{
  background:lightgreen;
}

footer{
  clear:both;
}
#gallery {
  height:1350px;
}
#gallery li {
  background:aqua;

}

2 Answers

Damien Watson
Damien Watson
27,419 Points

Hi Deepak,

Looks like you've messed up your styles a little. This is written in the order of the steps. First img, then gallery, gallery li and a final adjustment to the gallery li.

img {
  max-width: 100%;
}

#gallery {
  margin:0;
  padding:0;
  list-style:none;
}

#gallery li {
  float:left;
  margin:2.5%;
  width:45%;
  color:#f00;
  background-color:#0f0;
}
Modou Sawo
Modou Sawo
13,141 Points

Hi Deepak! Try setting the images' maximum to 100%

img {
    max-width: 100%;
}