JavaScript JavaScript and the DOM Responding to User Interaction Event Delegation

Adil Ismaaeel
Adil Ismaaeel
3,881 Points

Adding condition that changes element

Im not sure how to filter all but 'INPUT'. Please help

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

section.addEventListener('click', (e) => {
  if(e.target.tagName == 'INPUT') {
  e.target.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>

2 Answers

Elena Man
Elena Man
19,226 Points

Your answer seems alright. I tried out this code and it worked:

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

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

});

I think the issue here is that your other line of code is wrong. It's supposed to be

e.target.style.backgroundColor

and not

e.target.backgroundColor = 'rgb(255, 255, 0)';
Kevin Gates
Kevin Gates
14,700 Points

Use the Inequality (!=) operator instead of the equality (==). Learn more on Mozilla