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

Thomas Barkel
PLUS
Thomas Barkel
Courses Plus Student 7,222 Points
Posted by Thomas Barkel
Thomas Barkel
Courses Plus Student 7,222 Points

Help on Code Challenge

Unsure what I'm doing wrong, please assist.

Code Challenge is: In the following tasks you'll be required to select various elements on the index.html page. In the app.js file on line 1, select all links in the nav element and assign them to navigationLinks.

Error that I'm getting is: We expected 3 links not 8

js/app.js 
let navigationLinks = document.getElementsByTagName('A');
document.write(navigationLinks.length);
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

tobiastrinkler
seal-mask
.a{fill-rule:evenodd;}techdegree
tobiastrinkler
Full Stack JavaScript Techdegree Student 16,538 Points
tobiastrinkler
.a{fill-rule:evenodd;}techdegree
tobiastrinkler
Full Stack JavaScript Techdegree Student 16,538 Points

Hi Thomas, there are just couple minor things that prevent you from getting the challenge right.

First of all you we do not use Uppercase letters to select elements like ('A') instead use('a'). The second issue is, it is asking to select all a tags in the nav element. Therefore you could use an descendant selector like this : ('nav a') which gives you all a tags which are in an nav element. Lastly I would use querySelectorAll to return an HTML collection of all elements. Eventually, the code should look like this. Let me know it that helps!

let navigationLinks = document.querySelectorAll('nav a');
let galleryLinks;
let footerImages;
``
Robbie Thomas
Robbie Thomas
31,093 Points
Robbie Thomas
Robbie Thomas
31,093 Points

I hate how TreeHouse does these tricks on you, but I suppose that's something one should expect in the world of programming. I would have never thought you had to put two elements in there.