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 trialJessica Pehrson
1,648 PointsHELP
its saying i havent set maxium width but i have 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;
}
#gallery {
flex: auto;
max-width: 45%;
}
1 Answer
Jason Anders
Treehouse Moderator 145,860 PointsHi Jessica and Welcome to Treehouse.
I'm not sure why you are using an ID selector and flex.
The challenge wants you to add CSS that will allow all images to fill their parent element.
So, first off, you need to use the img
element selector to select all the images, and then you will need to set the width to 100%:
img {
max-width: 100%;
}
Keep Coding! :)
Greg Kaleka
39,021 PointsQuick correction - you need to set max-width to 100%, not width :)
Jason Anders
Treehouse Moderator 145,860 PointsThank-you for catching that Greg!
Sometimes my fingers type faster than my brain puts out. :) I made the correction in the answer.
Greg Kaleka
39,021 PointsHah don't we all?!
Stephen Chamroeun
1,228 PointsStephen Chamroeun
1,228 Pointsso you want to use this code here for all img's and it should work to go on to the next task
img { max-width: 100%; }