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 JavaScript and the DOM (Retiring) Responding to User Interaction Event Delegation

How do I "Add a condition that changes the background of the <input> elements only." ?

Hello!

I've hit a bit of a roadblock working on this challenge and I'm not sure what I'm doing wrong/right. I selected the 'input' tag with document.getElementsByTagName and set it equal to a variable, input. Then in my event handler I added a for loop to iterate through the input elements and change their background to black.

If anyone could give me some pointers I'd really appreciate it!

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

section.addEventListener('click', (e) => {
  e.target.style.backgroundColor = 'rgb(255, 255, 0)';
  for ( let i = 0; i < input.length; i++; ) { 
    input[i].style.backgroundColor = 'black';
  }
});
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>
HIDAYATULLAH ARGHANDABI
HIDAYATULLAH ARGHANDABI
21,010 Points

Hello Phaedra Ford, Actually you have done a great job you should repeat the same for the input as repeated for section[0]

3 Answers

Hey Phaedra Ford,

For this challenge, you can use the if statement and only target the Input element rather than selecting its all child elements.

Here's how you can achieve that:

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

section.addEventListener('click', (e) => {
  if(e.target.tagName === "INPUT"){
    e.target.style.backgroundColor = 'rgb(255, 255, 0)';
  }
});

Hope this helps!

Thank you so much for your help! The solution makes total sense. I think I also encountered an error because I used the color 'black' instead of the pre-chosen rgb color value - silly me.

Hey Phaedra Ford,

I am glad that you managed to fix it yourself, that's the part of learning.

Happy Coding!

HIDAYATULLAH ARGHANDABI
HIDAYATULLAH ARGHANDABI
21,010 Points

Hello Phaedra Ford, Actually you have done a great job you should repeat the same for the input as repeated for section[0]