Please help, anyone there to figure out the solution to this code error(display of undefined) in the selection menu...

Question

https://code.sololearn.com/WstAHwfAr5JW/?ref=app

Answer 1

Just change the event type for line 43 to "change" and it will fix it.

filterOption.addEventListener("change",filterTodo);

Answer 2

Try this;

function filterTodo(event){
const todos = todoList.children;
for(let i = 0; i < todos.length; i++) {
switch(event.target.value){
case "all":
todos[i].style.display = "flex";
break;
case "completed":
if(todos[i].classList.contains("completed")){
todos[i].style.display = "flex";
}
else{
todos[i].style.display = "none";
}
break;
case "uncompleted":
if(!todos[i].classList.contains("completed")){
todos[i].style.display = "flex";
}else{
todos[i].style.display = "none";
}
break;
}
}
}

Answer 3

Thanks alot "ChaoticDawg" it worked fine now,
well done bro👍🏼!

Answer 4

Samuel Mathew ya that was the 1 I was talking about that I missed. You need to add [i] after todos in the if statement, like all the others have for the index of todos.

Answer 5

Yeah! I just noticed that now...

Answer 6

Actually, you can just change this 1 line

const todos = Array.from(todoList.childNodes).slice(1);

Then it should work, but you still have a bug.

Answer 7

I see you changed it to my 1st post.
I missed 1 on line 104, so, if you're going to use this version make sure you fix it too.

Answer 8

Okay I got it now, thanks bro.

Answer 9

FYI, you need to make this function fire on item selected instead of the click. That should fix the behavior bug that it currently has.

Answer 10

Done, now it works fine, thanks bro...

Answer 11

When i looked just now, there isn't an error. Did you find the solution?

Answer 12

How can I do that, is there still bug in the code...