This doesn't work?

Question

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)';
  }
});

Answer 1 · 2019-01-01T02:00:50Z

January 1, 2019 2:00am

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')

Welcome to the Treehouse Community

Looking to learn something new?

Jason Pallone

Jason Pallone

This doesn't work?

1 Answer

Juan Lopez

Juan Lopez