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) Traversing the DOM Solution: Using nextElementSibling

tom Barkan
PLUS
tom Barkan
Courses Plus Student 3,062 Points
Posted by tom Barkan
tom Barkan
Courses Plus Student 3,062 Points

Down Button won't work

https://w.trhou.se/bsa0wrhi4x

app.js:35 Uncaught SyntaxError: missing ) after argument list

4 Answers

Steven Parker
Steven Parker
229,732 Points
Steven Parker
Steven Parker
229,732 Points

I see two issues:

  • the arguments on line 32 are in the wrong order
  • there's a stray extra close brace on lin 35

So, starting with line 31:

    if (nextLi) {
      ul.insertBefore(nextLi, li);  // instead of (li, nextLi)
    }
  }
// }  <-  removed extra close brace
Kat Stacy
Kat Stacy
41,658 Points
Kat Stacy
Kat Stacy
41,658 Points

I used all of the same code the teacher used but my down button is not working. Help please!

listUl.addEventListener('click', (event) => {
  if (event.target.tagName == 'BUTTON') { 
    if (event.target.className == 'remove') {
      let li = event.target.parentNode;
      let ul = li.parentNode;
      ul.removeChild(li);
    }
    if (event.target.className == 'up') {
      let li = event.target.parentNode;
      let prevLi = li.previousElementSibling;
      let ul = li.parentNode;
      if (prevLi) {
        ul.insertBefore(li, prevLi);
      }
    }
    if (event.target.className == 'down') {
      let li = event.target.parentNode;
      let nextLi = li.nextElementSibling;
      let ul = li.parentNode;
      if (nextLi) {
        ul.insertBefore(nextLi, li);
      }
    }
  }
});
Steven Parker
Steven Parker
229,732 Points
Steven Parker
Steven Parker
229,732 Points

The down buttons work for me. Perhaps the problem is in a different part of the code that is not shown here.

I recommend you start a fresh question, and make a snapshot of your workspace and post the link to it with your question.

Kat Stacy
Kat Stacy
41,658 Points
Kat Stacy
Kat Stacy
41,658 Points

Thank you Steven. I was able to figure it out. There was a problem with the HTML.