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 Traversing the DOM Event Delegation

Nicholas Espinosa
Nicholas Espinosa
5,981 Points

I thought I filtered out all but the 'INPUT' elements?

I used a conditional statement to filter out all but the 'INPUT' elements as was instructed in the video. The code seems to run properly, changing the background color of the 'input' elements when clicked (a different color than that of the 'div' element), but I am receiving an error message that tells me to 'filter out all but the 'INPUT' elements'. Is there a different way that I should be writing this section of code?

app.js
let section = document.getElementsByTagName('section')[0];

section.addEventListener('click', (e) => {
  if (event.target.tagName === 'INPUT') {
    e.target.style.backgroundColor = 'blue';
  } else {
  e.target.style.backgroundColor = 'rgb(255, 255, 0)';
  }
});
index.html
<!DOCTYPE html>
<html>
    <head>
        <title>JavaScript and the DOM</title>
    </head>
    <link rel="stylesheet" href="style.css" />
    <body>
        <section>
            <h1>Making a Webpage Interactive</h1>
            <p>JavaScript is an exciting language that you can use to power web servers, create desktop programs, and even control robots. But JavaScript got its start in the browser way back in 1995.</p>
            <hr>
            <p>Things to Learn</p>
            <ul>
                <li>Item One: <input type="text"></li>
                <li>Item Two: <input type="text"></li>
                <li>Item Three: <input type="text"></li>
                <li>Item Four: <input type="text"></li>
            </ul>
            <button>Save</button>
        </section>
        <script src="app.js"></script>
    </body>
</html>

1 Answer

Steven Parker
Steven Parker
216,891 Points

You've got two issues:

  • the code should change the background color (using the provided line) of only the input elements
  • you won't need an "else", nothing needs to be done for other elements