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 trialMike Carroll
743 PointsCSS for h2 media query doesn't seem to be working...is my code right here? Thanks,
After hitting "Recheck Work", it keeps saying "Bummer!....did you change the h2 font-size to 1.75em for devices larger than 660px?
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) {
h1 {
font-size: 2.5em;
}
@media screen and (min-width: 660px) {
h2 {
font-size: 1.75em;
}
}
3 Answers
dman10345
7,109 PointsLooks like you have messed up your curly braces here a little bit. You don't have even amounts in your media queries. For every { there should be a } to match it.
@media screen and (min-width: 480px) {
h1 {
font-size: 2.5em;
}
@media screen and (min-width: 660px) {
h2 {
font-size: 1.75em;
}
}
Notice how there are 4 { but only 3 }. Simple fix just add a } after your first query to close the entire block. This is easier to see if you format them correctly.
@media screen and (min-width: 480px) {
h1 {
font-size: 2.5em;
} /*<--- This one closes your h1 selector */
} /*<--- This one closes your media query */
@media screen and (min-width: 660px) {
h2 {
font-size: 1.75em;
} /*<--- This one closes your h2 selector */
} /*<--- This one closes your media query */
Chad Helmer
14,825 PointsAdd the closing bracket to your first media query and should be good to go
Mike Carroll
743 PointsThank you Chad and Dylan, I did need to add the closing bracket.