difficult passing the code challenge

Question

cant get the 3 links, i keep on getting 5 links

js/app.js

let navigationLinks;
let galleryLinks;
let footerImages;
const myLinks = document.querySelectorAll("li");
for ( let i = 0; i < 3; i += 1) {
  navigationLinks+= myLinks[i];
}

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>

Answer 1 · 2018-08-18T16:52:55Z

August 18, 2018 4:52pm

I'm baffled by the error message "Bummer: Was expecting 3 links not 75". I would expect that adding elements to an undefined variable would simply fail, not give 75 links.

But you won't need a loop, the "querySelectorAll" method will do what you need with the right argument.

Also, remember that links are implemented with anchor ("a") elements, an "li" is a list item.

Answer 2 · 2018-08-18T20:14:44Z

August 18, 2018 8:14pm

Try thinking about understanding specificity. The for loop is not needed but I could see what you were trying to do. Here are 2 ways you could achieve what you want.

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

You could also try adding a class to the parent ul. then using querySelectorAll with the class Like this

html

<ul class="navlinks">

app.js

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

I hope that helps.

Welcome to the Treehouse Community

Looking to learn something new?

Abdul Azeez

Abdul Azeez

difficult passing the code challenge

2 Answers

Steven Parker

Steven Parker

Joshua Ticzon

Joshua Ticzon

Steven Parker

Steven Parker