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 Interactive Web Pages with JavaScript Traversing and Manipulating the DOM with JavaScript Adding Multiple Event Listeners

get stuck at adding-multiple-event-listeners

//Select select box var navigationSelect = document.getElementById("nav");

//Navigate to URL when select box is changed var navigateToValue = function() { window.location = this.value; }

//Send analytics data var sendAnalytics = function() { //Placeholder
}

navigationSelect.addEventListener("onchange" ,navigateToValue); navigationSelect.addEventListener("onchange" ,sendAnalytics);

app.js
//Select select box
var navigationSelect = document.getElementById("nav");

//Navigate to URL when select box is changed
var navigateToValue = function() {
  window.location = this.value;
}

//Send analytics data
var sendAnalytics = function() {
  //Placeholder  
}

navigationSelect.addEventListener("onchange" ,navigateToValue);
navigationSelect.addEventListener("onchange" ,sendAnalytics);
index.html
<!DOCTYPE html>
<html>
  <body>
    <select id="nav">
      <option value="index.html">Home</option>
      <option value="about.html">About</option>
    </select>
    <h1>Home</h1>
    <script src="app.js"></script>
  </body>
</html>
about.html
<!DOCTYPE html>
<html>
  <body>
    <select id="nav">
      <option value="index.html">Home</option>
      <option value="about.html">About</option>
    </select>
    <h1>About</h1>
    <script src="app.js"></script>
  </body>
</html>

3 Answers

rydavim
rydavim
18,814 Points

You're very close, you just have a small syntax issue. When adding your event listeners, you want to bind it to 'change' instead of 'onchange'. That should do it! Happy coding! :)

Bob McCarty
PLUS
Bob McCarty
Courses Plus Student 16,618 Points

Try

navigationSelect.addEventListener("change", navigateToValue);
navigationSelect.addEventListener("change", sendAnalytics);

Thank you rydavim and bob I have passed :)