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

Amber Clark
Amber Clark
10,001 Points
Posted by Amber Clark
Amber Clark
10,001 Points

Not sure what the interpreter wants. I have two solutions that worked on my browser. Help!

These are the two solutions that have worked on my browser: 1) let nav = document.querySelector('nav'); let navigationLinks = nav.querySelectorAll('li'); let galleryLinks; let footerImages;

2) edit the html so that the <li> that I want to grab have a className of navLink. <li class='navLink'><a href="index.html" class="selected">Portfolio</a></li> <li class='navLink'><a href="about.html">About</a></li> <li class='navLink'><a href="contact.html">Contact</a></li>

app.js: let navigationLinks = document.querySelectorAll('.navLink'); let galleryLinks; let footerImages;

Both of these solutions return a collection of the 3 <li> tags that the challenge is requesting. I can't see another way to do it and clearly there is a different way the interpreter is looking for. Any ideas are appreciated.

Thanks!

js/app.js 
let navigationLinks = document.querySelectorAll('.navLink');
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 class='navLink'><a href="index.html" class="selected">Portfolio</a></li>
          <li class='navLink'><a href="about.html">About</a></li>
          <li class='navLink'><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

Steven Parker
Steven Parker
231,271 Points
Steven Parker
Steven Parker
231,271 Points

Don't change the HTML, the purpose of the exercise is to select things as they are.

Your first attempt was close, it would have selected the list items in the nav element. But the instructions ask you to "select all links in the nav element", and links are represented by anchor ("a") elements.