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

Posted by Mattis Knieling
Mattis Knieling
1,536 Points

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

Rabin Gharti Magar
seal-mask
PLUS
.a{fill-rule:evenodd;}techdegree seal-36
Rabin Gharti Magar
Front End Web Development Techdegree Graduate 20,928 Points
Rabin Gharti Magar
.a{fill-rule:evenodd;}techdegree seal-36
Rabin Gharti Magar
Front End Web Development Techdegree Graduate 20,928 Points

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!

Mattis Knieling
Mattis Knieling
1,536 Points
Mattis Knieling
Mattis Knieling
1,536 Points

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