Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

JavaScript Interacting with the DOM Responding to Events Adding an Event Listener

Rashad Nelson
Rashad Nelson
5,770 Points

Please review my code

I get it for the most part. we are pulling the warning div into the variable warning. We create the listener and add function. Is the code im running in the call back function wrong. If i wanted to use the css selector of ID would #warning{background-color: red} work as well or since we are working with the DOM better to user the style.element. option.

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

<!DOCTYPE html>
        <title>Adding an Event Listener</title>
    <link rel="stylesheet" href="style.css" />
        <div id="warning">
            Warning: My background should be red!
        <button id="makeItRed">Make It Red!</button>
        <script src="app.js"></script>

1 Answer

Steven Parker
Steven Parker
217,639 Points

By naming the parameter of your event handler "warning", it "shadows" (prevents access to) the global "warning" variable you defined in task 2.

Give the parameter another name ("event" or "e" are both commonly used) and the handler will work as expected.