I cannot answer the code challenge. It says one or more of the buttons does not affect

Question

Here is my code:

var list = document.getElementsByTagName('ul')[0];

list.addEventListener('click', function(e) { if (e.target.tagName == 'BUTTON') { let li = e.target.parentNode; let ul = li.parentNode; if (e.target.previousElementSibling == 'P') { e.target.previousElementSibling.className = 'highlight'; }

} });

app.js

var list = document.getElementsByTagName('ul')[0];

list.addEventListener('click', function(e) {
  if (e.target.tagName == 'BUTTON') {
    let li = e.target.parentNode;
    let ul = li.parentNode;
    if (e.target.previousElementSibling == 'P') {
      e.target.previousElementSibling.className = 'highlight';
    }

  }
});

index.html

<!DOCTYPE html>
<html>
    <head>
        <title>JavaScript and the DOM</title>
    </head>
    <link rel="stylesheet" href="style.css" />
    <body>
        <section>
            <h1>Making a Webpage Interactive</h1>
            <p>Things to Learn</p>
            <ul>
                <li><p>Element Selection</p><button>Highlight</button></li>
                <li><p>Events</p><button>Highlight</button></li>
                <li><p>Event Listening</p><button>Highlight</button></li>
                <li><p>DOM Traversal</p><button>Highlight</button></li>
            </ul>
        </section>
        <script src="app.js"></script>
    </body>
</html>

Answer 1 · 2020-12-28T11:28:05Z

December 28, 2020 11:28am

Hi there,

No need to add another if statement inside the previous If statement. You're code should look like so:

var list = document.getElementsByTagName('ul')[0];

list.addEventListener('click', function(e) {
  if (e.target.tagName == 'BUTTON') {
    e.target.previousElementSibling.className = 'highlight';
  }
});

Hope this helps, Shaun

Welcome to the Treehouse Community

Looking to learn something new?

xajx

xajx

I cannot answer the code challenge. It says one or more of the buttons does not affect

1 Answer

Shaun Kelly

Shaun Kelly