JavaScript JavaScript and the DOM Responding to User Interaction Event Delegation

Jason Pallone
Jason Pallone
8,979 Points

This doesn't work?

So I ran the code in my workspace and ran the script on a browser, and it worked exactly like they want it to. It changed the color only for the LI elements? Why won't this pass?

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>
app.js
let section = document.getElementsByTagName('section')[0];


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

1 Answer

Juan Lopez
Juan Lopez
Full Stack JavaScript Techdegree Student 15,330 Points

You're close but the challenge is asking for the "input elements" to trigger the background change. Currently on your code if you click anywhere on the "LI" it changes the background of the whole "LI" and it only wants the input box to change it background color.

if (e.target.tagName === 'INPUT')