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

JavaScript JavaScript and the DOM (Retiring) Responding to User Interaction Adding an Event Listener

Robert Lawrence
Robert Lawrence
2,599 Points
Posted by Robert Lawrence
Robert Lawrence
2,599 Points

my solution works, but the tool says it doesn't

var warning = document.getElementById("makeItRed"); var button = warning; button.addEventListener("click", () => { var getButton = document.getElementById('warning'); getButton.style.backgroundColor = "red"; } );

it works, but it's not letting me pass

app.js 
var warning = document.getElementById("makeItRed");
var button = warning;
button.addEventListener("click", () => {
    var getButton = document.getElementById('warning');
  getButton.style.backgroundColor = "red";
} );
index.html 
<!DOCTYPE html>
<html>
    <head>
        <title>Adding an Event Listener</title>
    </head>
    <link rel="stylesheet" href="style.css" />
    <body>
        <div  id="warning">
            Warning: My background should be red!
        </div>
        <button id="makeItRed">Make It Red!</button>
        <script src="app.js"></script>
    </body>
</html>

1 Answer

KRIS NIKOLAISEN
KRIS NIKOLAISEN
54,971 Points
KRIS NIKOLAISEN
KRIS NIKOLAISEN
54,971 Points

You didn't follow the instructions. Any code provided needs to stay as is unless you are specifically asked to change. The first task is: Select the button with the ID makeItRed and assign it to the button variable. You have the code for this but overwrote var warning instead. It should be:

var warning = document.getElementById("warning");
var button = document.getElementById("makeItRed");

You have the second task ok. 

button.addEventListener("click", () => {
} );

The third task is: in the button click handler, apply a background color of "red" to the warning div. There is no need to create a new variable. If you check the formatting you are specifically asked to use the warning div.

warning.style.backgroundColor = "red";