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

CSS CSS Basics (2014) Enhancing the Design With CSS Media Queries

Posted by Obe Juarez
Obe Juarez
6,357 Points

Why wont my code work?

this is my code @media only screen and (max-width: 1020px) { .main-header { background-color: tomato; color: white; } } i do't get why it wont work

style.css 
@media only screen and (max-width: 1020px) {
    .main-header {
    background-color: tomato;
    color: white;
  }
}
index.html 
<!DOCTYPE html>
<html>
  <head>
    <title>Lake Tahoe</title>
    <link rel="stylesheet" href="page.css">
    <link rel="stylesheet" href="style.css">
  </head>
  <body> 
    <header id="top" class="main-header">
      <span class="title">Journey Through the Sierra Nevada Mountains</span>
      <h1 class="main-heading">Lake Tahoe, California</h1>
    </header>

        <div class="primary-content">
            <p class="intro">
                Lake Tahoe is one of the most breathtaking attractions located in California. It's home to a number of ski resorts, summer outdoor recreation, and tourist attractions. Snow and skiing are a significant part of the area's reputation.
            </p>
            <a class="callout" href="#more">Find out more</a>
        </div><!-- End .primary-content -->
  </body>
</html>

3 Answers

Jamie Reardon
seal-mask
.a{fill-rule:evenodd;}techdegree seal-36
Jamie Reardon
Treehouse Project Reviewer
Jamie Reardon
.a{fill-rule:evenodd;}techdegree seal-36
Jamie Reardon
Treehouse Project Reviewer

Hi, I've tested your code and it appears to work fine for me when using the html document and style.css, do you have any other code in your style.css?

Unsubscribed User
seal-mask
.a{fill-rule:evenodd;}techdegree
Unsubscribed User
Front End Web Development Techdegree Student 33,900 Points
Unsubscribed User
.a{fill-rule:evenodd;}techdegree
Unsubscribed User
Front End Web Development Techdegree Student 33,900 Points

Hm, I don't know why it works in Jamies code but for me the challenge only accepts:

@media (max-width: 1020px) {
    .main-header {
      background-color: tomato;
      color: white;
  }
}

No idea why though...

It also does not accept

@media screen and (max-width: 1020px) {
    .main-header {
      background-color: tomato;
      color: white;
  }
}

which should also be valid normally...

Maybe someone else can shed a light on that. I am curious as well...

Jamie Reardon
seal-mask
.a{fill-rule:evenodd;}techdegree seal-36
Jamie Reardon
Treehouse Project Reviewer
Jamie Reardon
.a{fill-rule:evenodd;}techdegree seal-36
Jamie Reardon
Treehouse Project Reviewer

Ah sorry, didn't see that this was a code challenge attempt since you are using more than the default value for the media query.

Challenge 1 accepts:

@media (max-width: 1020px) {

  .main-header {
    background-color: tomato;
    color: white;
  }

}

Challenge 2 accepts:

@media (max-width: 768px) {

  .title {
    font-size: 1.4rem;
  }

  h1 {
    font-size: 5rem;
  }

}

You are using the only screen and the and operator when the challenge only wants you to use the default value (all).