CSS CSS Basics (2014) Basic Selectors ID and Class Selectors

Targeting class attribute

Need to change H1 element "main heading" to class attribute what am i doing wrong ?

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 class="main-header">
      <span>Journey through the Sierra Nevada Mountains</span>
      <h1  .main-heading 
          {font-size: 
          72 px;}
          >Lake Tahoe, California</h1>
    </header>
    <p>
      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 href="#">Find out more</a>
    <div>
      <h2>Check out all the Wildlife</h2>
      <p>
        As spawning season approaches, the fish acquire a humpback and protuberant jaw. After spawning, they die and their carcasses provide a feast for gatherings of mink, bears, and Bald eagles.
      </p>
      <a href="#">See the Wildlife</a>
    </div>
  </body>
</html>
style.css
/* Complete the challenge by writing CSS below */
.main-header {
background-color: orange;
}

2 Answers

rydavim
rydavim
18,748 Points

You're on the right track, just remember to separate your html from your css for this challenge.

<header class="main-header"> <!-- Nice job! -->

<!-- For this step, adding a class to this html should look more like what you did above. -->
 <!-- No CSS here, set the font size in the other file. -->
<h1 .main-heading {
    font-size: 72 px; 
    }
>Lake Tahoe, California</h1>
.main-header { /* Perfect! */
  background-color: orange;
}

.main-heading { /* Try putting the code you wrote in the HTML above in the CSS file instead. */
  font-size: 72 px; /* Otherwise, it looks good! */
}

Pretty much all of your code looks good, so if you untangle it back out you should pass with flying colors. Let us know if you run into any trouble, though. Happy coding!

Daniel Barros
Daniel Barros
Front End Web Development Techdegree Graduate 23,588 Points

Hi Briona,

In your html <h1> tag you included the css declaration. The declaration should either appear in a separate stylesheet or in within the <head> element in between <style> tags. You could also do it with the style attribute.

<head>
    <title>Lake Tahoe</title>
    <link rel="stylesheet" href="page.css">
    <link rel="stylesheet" href="style.css">
    <style>
        .main-heading {
            font-size: 72 px;
        }
     </style>
  </head>

After this, just add the class '.main-heading to your h1 element.

 <h1  class="main-heading">Lake Tahoe, California</h1>

This should do it. If not, just let me know and I'll give it another go! Cheers,

Daniel

Mod Edit: Fixed the code formatting, and change this comment to an answer so it may be voted on or marked as best. See comment for how-to on fancy-formatting for code.

rydavim
rydavim
18,748 Points

When posting code, there is some markdown you can use to make it more readable. If you wrap your code in three back-ticks you'll get special formatting, and you can optionally add the language for fancy syntax highlighting.

```language

Your code here!

```

// Your code here!

I also went ahead and changed your comment to an answer so that users can give you credit for it. Thanks for helping out in the community!