what am i doing wrong ?

Question

can anyone help please?

app.js

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

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

section.addEventListener('click', (imput) => {
imput.target.style.backgroundColor = 'rgb(0, 0, 0)';
});

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>

Answer 1 · 2020-07-02T08:18:17Z

July 2, 2020 8:18am

In order to target just the input elements, you have to add a if statement

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!

Welcome to the Treehouse Community

Looking to learn something new?

Alexandru Pescaru

Alexandru Pescaru

what am i doing wrong ?

1 Answer

Daniel Jong

Daniel Jong