Can't find the sollution for this Challenge Task

Question

Do I really need a loop for this?

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('input')[0];

section.addEventListener('click', (bg) => {
  for ( let i=0;  i < section.length; i++) {
  section[i].target.style.backgroundColor = 'rgb(255, 255, 0)';
  }
});

Answer 1 · 2017-07-02T03:46:49Z

July 2, 2017 3:46am

No, you don't need a loop.

All you need is a test to be sure you only change the color on the correct type of elements (the instructions say "text input elements").

Welcome to the Treehouse Community

Looking to learn something new?

Paulo Oliveira

Paulo Oliveira

Can't find the sollution for this Challenge Task

1 Answer

Steven Parker

Steven Parker

No, you don't need a loop.