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![](https://ecs-static.teamtreehouse.com/assets/views/marketing/shared/community-banner-white-47072046c51352fe6a69f5e691ff5700b28bb11d45197d7bdf066d9ea3f72d0c.webp)
![Hanwen Zhang](https://uploads.teamtreehouse.com/production/profile-photos/11590826/micro_Picture.jpg)
Hanwen Zhang
20,084 PointsWhy my code is incorrect here? Shouldn't use 'p' to reference paragraph?
Thanks
var list = document.getElementsByTagName('ul')[0];
let p = document.querySelectorAll ('p');
list.addEventListener('click', function(e) {
if (e.target.tagName == 'BUTTON') {
p.previousElementSibling.className = 'highlight'
}
});
<!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>
1 Answer
![Peter Vann](https://uploads.teamtreehouse.com/production/profile-photos/8935412/micro_MaiTaiPeteGravatar.jpg)
Peter Vann
36,426 PointsHi Hanwen!
No, you don't need to reference any p tags, since you are using DOM sibling transversal.
This passes:
var list = document.getElementsByTagName('ul')[0];
list.addEventListener('click', function(e) {
if (e.target.tagName == 'BUTTON') {
e.target.previousElementSibling.className += " highlight";
}
});
Note that the strategy/logic here is to target all the buttons and then add the highlight class to each button's previousElementSibling, which are, in each case, paragraphs.
The reason to learn to do DOM transversal this way is that, in certain cases, some elements are more difficult to target than others. In other words, sometimes it's easier to target an easy-to-target element (because it has an easily identifiable id, for example) and the transverse up or down the DOM structure to find your real intended target, because perhaps that element has no identifying id or class.
It's analogous to calling or texting someone whose phone number you do have to try to get ahold of someone whose number you don't have/can't remember (but the person you did call knows how to get in touch with the real person you need to call/text).
Does that make sense?
I hope that helps.
Stay safe and happy coding!
Hanwen Zhang
20,084 PointsHanwen Zhang
20,084 PointsHi Peter,
I understood now, Thank you so much for your detailed explanation.