is my code right ??

Question

can some one please help me over come this challenging task? it says:In the listener that has been added to the section element, ensure that the text input elements are the only children that trigger the background-changing behavior. and this is my code i do not know what I am doing wrong?!!

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 · 2017-07-22T12:55:20Z

July 22, 2017 12:55pm

It looks like this code is testing for the wrong elements.

As you pointed out, the instructions say "ensure that the text input elements are the only children that trigger the background-changing behavior." But the if statement is testing for list items ("li" elements).

Also, don't forget that tag name properties are stored in upper case.

Welcome to the Treehouse Community

Looking to learn something new?

yousif alyousif

yousif alyousif

is my code right ??

1 Answer

Steven Parker

Steven Parker

It looks like this code is testing for the wrong elements.