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 Text Shadows and Box Shadows

Lacey Jayne
Lacey Jayne
3,519 Points

Why is it saying to add a comma?, the code is working!?

Why is it saying to add a comma?, the code is working!?

style.css
/* Complete the challenge by writing CSS below */
.main-heading{
  text-shadow: 0 0 5px #be7b31}
.title {
  text-shadow: 1px 3px 0px #e59740
}
.main-header{
  box-shadow: 0px 2px 15px #aaa;
  box-shadow: inset 0 0 60px 5px firebrick;
}
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 -->

        <footer class="main-footer">
            <p>All rights reserved to the state of <a href="#">California</a>.</p>
            <a href="#top">Back to top &raquo;</a>
        </footer>
  </body>
</html>

3 Answers

Jeff Lemay
Jeff Lemay
14,268 Points

The code is working (nothing is broken) but you only have the second box-shadow.

Without the comma, you are overwriting the first one when you declare the second (cascading styles). You want to have a box-shadow with both outset and inset which is done by combining the two with a comma.

  box-shadow: ---;
Jonathan Grieve
Jonathan Grieve
Treehouse Moderator 91,253 Points

I've been trying that same solution on the code challenge. I'm sure it's correct but my challenge isn't letting it pass. Very odd. :-)

Matt F.
Matt F.
9,518 Points

You should remove the second 'box-shadow' and replace the preceding semicolon with a comma, like so:

.main-header{
  box-shadow: ---;
}

Because you use a comma in between multiple changes of the same property, in this case the box-shadow. The way you have it, the second line of box shadow is (or should be) overwriting the first. The first line of box shadow in this code is very subtle, so I wouldn't be surprised if it wasn't registering and you didn't notice.