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 trialBirthe Vandermeeren
17,146 PointsMy solution to Guil's challenge of hiding the non-working buttons
I've created a function hideNonWorkingButtons that first loops through all buttons in the list to set their visibility to "visible".
Then, it sets the visibility of the up button of the first item in the list and the visibility of the down button of the last item in the list to "hidden".
I'm calling this function after all the buttons are created so the right buttons are hidden on the start screen. Then I'm calling it again at the end of the listUl.addEventListener, so it hides the correct buttons when one of the up, down, or remove buttons are clicked. And again at the end of the addItemButton.addEventListener, so it hides the correct buttons when an item is added to the list.
Hope this makes sense. If you have any questions, I'd be happy to clarify.
Here's my the link to my workspace: workspace with function to hide the non-working buttons
1 Answer
Mindaugas Kunevicius
11,273 PointsNice! Just you don't need to define let allListItems
on line 39 as you already have const lis
on line 12. And it itches for me to somehow use the defined const firstListItem
and const lastListItem
in the solution, but I didn't manage yet as well...
I like your solution, but I have simplified your function a bit and added comments: My Workspace
Update: I found the way to use firstElementChild
and lastElementChild
to traverse instead of counting items in the array :) Updated Workspace
Birthe Vandermeeren
17,146 PointsBirthe Vandermeeren
17,146 PointsYou're right, Mindaugas, about the allListItems and lis, and right, firstElementChild, smart. Thanks for commenting!