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) Getting a Handle on the DOM Selecting Multiple Elements

Michael Lawinger
Michael Lawinger
33,581 Points
Posted by Michael Lawinger
Michael Lawinger
33,581 Points

Iterating Through Colors With Query Selector

The the coding challenge question that I'm struggling with is: We have some JavaScript code that will cycle over list items and apply colors from an array called colors. The code will apply the first color to the first list item, the second color to the second list item and so on. But the code is not complete. On line 1 of app.js, set the variable listItems to refer to a collection. The collection should contain all list items in the unordered list element with the id of rainbow.

My code is below

index.html 
<!DOCTYPE html>
<html>
  <head>
    <title>Rainbow!</title>
  </head>
  <body>
    <ul id="rainbow">
      <li>This should be red</li>
      <li>This should be orange</li>
      <li>This should be yellow</li>
      <li>This should be green</li>
      <li>This should be blue</li>
      <li>This should be indigo</li>
      <li>This should be violet</li>
    </ul>
    <script src="js/app.js"></script>
  </body>
</html>
js/app.js 
let listItems = document.querySelectorAll('ul li#rainbow');
const colors = ["#C2272D", "#F8931F", "#FFFF01", "#009245", "#0193D9", "#0C04ED", "#612F90"];

for(var i = 0; i < colors.length; i =+ 1) {
  listItems[i].style.color = colors[i];    
}

2 Answers

Gerald Wells
Gerald Wells
12,763 Points

It wants you to create an array of the items in the unordered list. To do this you should just be able to:

let listItems = document.querySelectorAll("li");

which will create an array of all the list items for the for loop to iterate through.

Michael Lawinger
Michael Lawinger
33,581 Points
Michael Lawinger
Michael Lawinger
33,581 Points

So basically I was right but just being too specific?

Sebastian Velandia
Sebastian Velandia
24,676 Points
Sebastian Velandia
Sebastian Velandia
24,676 Points

listItems = document.querySelectorAll("li");