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) Making Changes to the DOM Getting and Setting Text with textContent and innerHTML

Agnieszka Niemiec
Agnieszka Niemiec
7,359 Points
Posted by Agnieszka Niemiec
Agnieszka Niemiec
7,359 Points

The code is not working

I wrote the same code as in the video and the button is not clickable.

KRIS NIKOLAISEN
KRIS NIKOLAISEN
54,967 Points

Can you post your code?

Agnieszka Niemiec
Agnieszka Niemiec
7,359 Points
Agnieszka Niemiec
Agnieszka Niemiec
7,359 Points

//app.js

const input = document.querySelector('input'); const p = document.querySelectorAll('p.description'); const button = document.querySelector('button');

button.addEventListener('click', () => { p.innerHTML = input.value + ':';
});

<!DOCTYPE html> <html> <head> <title>JavaScript and the DOM</title> <link rel="stylesheet" href="css/style.css"> </head> <body> <h1 id="myHeading">JavaScript and the DOM</h1> <p>Making a web page interactive</p>

<p class="description">Things that are purple:</p>
<input type="text" class="description">
<button class="description">Change list description</button>
<ul>
  <li>grapes</li>
  <li>amethyst</li>
  <li>lavender</li>
  <li>plums</li>
</ul>
<script src="app.js"></script>

</body> </html>

Thank you :)

1 Answer

Steven Parker
Steven Parker
229,644 Points
Steven Parker
Steven Parker
229,644 Points

The "querySelectorAll" function returns an element collection, which is what you store in "p". But later in the event handler, you try to assign the "innerHTML" property of "p", which collections don't have.

If you use the "querySelector" function instead, it will select the first single element and the handler will work.