Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

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

I did it, but in a other way... and it was worng

I just dont get it... i did it in 2 different way and both were wrong .

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

section.eventListener('click', (e)) {
    if(event.target.tagName = 'INPUT'){
      e.target.style.backgroundColor = 'yellow';
  };
}
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

Hey Mattis Knieling,

I have found a couple of errors on your code. I will list them down:

  1. It should be addEventListener instead of eventListener
  2. Remove the last closing bracket from function parameter, it should be like this (e)
  3. You are missing an arrow from function after defining parameters. So, add this arrow => right after defining (e).
  4. Inside a conditional statement parameter you should write e instead of event as that's what you have defined inside function parameter
  5. Instead of single equal add a strict equality operator ===
  6. You have changed the backgroundColor to yellow but you should just leave as it was rgb(255, 255, 0).
  7. And finally right after the closing curly brace add a closing bracket along with a semicolon

Here's the final version:

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

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

Hope this helps!

yeah, thanks .. i've got it.... i was just a bit frustrated and wrote some damn bad code .. thanks