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

Rashid Mikati
Rashid Mikati
2,163 Points
Posted by Rashid Mikati
Rashid Mikati
2,163 Points

I can't figure this out

I tried putting a class to each link and selecting the class but it didn't work. Can anyone help me out?

js/app.js 
let navigationLinks = document.getElementsByClassName("selected");
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" class='selected'>About</a></li>
          <li><a href="contact.html" class='selected'>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>
Rashid Mikati
Rashid Mikati
2,163 Points
Rashid Mikati
Rashid Mikati
2,163 Points

The question is to select all links in the navigation element

3 Answers

Jason Welsh
seal-mask
.a{fill-rule:evenodd;}techdegree seal-36
Jason Welsh
Treehouse Project Reviewer
Jason Welsh
.a{fill-rule:evenodd;}techdegree seal-36
Jason Welsh
Treehouse Project Reviewer

Change your class='selected' in your HTML to have double quotes first: 

class="selected"

and then use the getElementsByClassName() or querySelectorAll() to select them the way you have

:)

Jason Welsh
seal-mask
.a{fill-rule:evenodd;}techdegree seal-36
Jason Welsh
Treehouse Project Reviewer
Jason Welsh
.a{fill-rule:evenodd;}techdegree seal-36
Jason Welsh
Treehouse Project Reviewer

Rashid, I see you commented that it didn't work, so just in case you haven't gotten it sorted yet, I created a codepen to show how I changed the html attribute to use double quotes, and then I loop over the navigationLinks node list with a for loop and turn their text color to the ever lovely, "tomato". Hope this helps! https://codepen.io/anon/pen/KqPOvq

Jeff McDivitt
Jeff McDivitt
23,970 Points
Jeff McDivitt
Jeff McDivitt
23,970 Points

You are looking to select the links in the navigation element, I did this with querySelectorAll

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