Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

JavaScript JavaScript and the DOM (Retiring) Responding to User Interaction Event Delegation

Ali Sh
Ali Sh
2,026 Points
Posted by Ali Sh
Ali Sh
2,026 Points

what is wrong with my code?

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.

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

2 Answers

Antonio De Rose
Antonio De Rose
20,885 Points
Antonio De Rose
Antonio De Rose
20,885 Points

because, the question states, if not for input type, let not the background change, and the condition, you are giving is for the LI, approach is fine, just change the condition to check for INPUT.

Ali Sh
Ali Sh
2,026 Points
Ali Sh
Ali Sh
2,026 Points

Sorry but i still can't understand.

Antonio De Rose
Antonio De Rose
20,885 Points
Antonio De Rose
Antonio De Rose
20,885 Points

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.

what does the question say - it says, text input elements are the only children, that trigger, the change behaviour

if that is what the question ask, you have actioned the task, for the child LI, to change the behaviour, that is wrong against the question.

now you have to approach, find how would you do that to the INPUT element, rather than the LI element case sensitive too

if (e.target.TagName == 'li') 
change to
if (e.target.TagName == '?????') {