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 Practice Selecting Elements

Posted by Zachary Eckstein
Zachary Eckstein
9,901 Points

How to only assign the <li> in <nav> to my variable and not the other <li> in other places on the page?

I am trying to only get the <li> in the <nav> section using querySelectorAll. They don't all have a common id, should I just go give them one to make it easier? Or am I supposed to be doing something else?

js/app.js 
let navigationLinks = document.querySelectorAll('li');#i want to do like nav.li to go into nav and get all the li only in there or something
let galleryLinks;
let footerImages;
index.html 
<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8">
    <title>Nick Pettit | Designer</title>
    <link rel="stylesheet" href="css/normalize.css">
    <link href='http://fonts.googleapis.com/css?family=Changa+One|Open+Sans:400italic,700italic,400,700,800' rel='stylesheet' type='text/css'>
    <link rel="stylesheet" href="css/main.css">
    <link rel="stylesheet" href="css/responsive.css">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
  </head>
  <body>
    <header>
      <a href="index.html" id="logo">
        <h1>Nick Pettit</h1>
        <h2>Designer</h2>
      </a>
      <nav>
        <ul>
          <li><a href="index.html" class="selected">Portfolio</a></li>
          <li><a href="about.html">About</a></li>
          <li><a href="contact.html">Contact</a></li>
        </ul>
      </nav>
    </header>
    <div id="wrapper">
      <section>
        <ul id="gallery">
          <li>
            <a href="img/numbers-01.jpg">
              <img src="img/numbers-01.jpg" alt="">
              <p>Experimentation with color and texture.</p>
            </a>
          </li>
          <li>
            <a href="img/numbers-02.jpg">
              <img src="img/numbers-02.jpg" alt="">
              <p>Playing with blending modes in Photoshop.</p>
            </a>
          </li>
        </ul>
      </section>
      <footer>
        <a href="http://twitter.com/nickrp"><img src="img/twitter-wrap.png" alt="Twitter Logo" class="social-icon"></a>
        <a href="http://facebook.com/nickpettit"><img src="img/facebook-wrap.png" alt="Facebook Logo" class="social-icon"></a>
        <p>&copy; 2016 Nick Pettit.</p>
      </footer>
    </div>
  <script src="js/app.js"></script>
  </body>
</html>

2 Answers

Joe Nguyen
Joe Nguyen
25,189 Points

You don't really have to put much in there to select all of that you could do nav a or nav ul li a like this

 let navigationLinks = document.querySelectorAll("nav a"); // or ("nav ul li a")
let galleryLinks;
let footerImages;
Dhanuka Anushanga
Dhanuka Anushanga
13,347 Points
Dhanuka Anushanga
Dhanuka Anushanga
13,347 Points

This code selects all the list items of unordered list within the nav element.

let navigationLinks = document.querySelectorAll("nav ul li");

But, the code challenge asked you to select all the links in the nav element. So, the code should look like this.

let navigationLinks = document.querySelectorAll("nav ul li a");