JavaScript JavaScript and the DOM Getting a Handle on the DOM Practice Selecting Elements

Sunny Balani
Sunny Balani
6,615 Points

Why does document.querySelectorAll('nav ul'); not work

it keeps saying i only selected one link

js/app.js
let navigationLinks = document.querySelectorAll('header nav');
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>

1 Answer

Dane Parchment
MOD
Dane Parchment
Treehouse Moderator 9,594 Points

Your code is actually doing exactly what it is supposed to do.

let navigationLinks = document.querySelectorAll('header nav');

There is only one element that is a descendent of header and is a nav. However, you are also targeting the wrong thing. We are looking to get all of the links that can be found under any nav.

So what does that mean? We need to get every link that can be found under any nav. So that means we need to find all the a tags that are descendants of a nav tag.

So it should look like this:

let navigationLinks = document.querySelectorAll('nav a');