Internal review

[yersel500]

I can improve the next part of my codes:

to-do-list/src/index.js

Lines 25 to 40 in 659775d

	ulContainer.addEventListener('click', (e) => {
	const index = Number(e.target.parentElement.classList[0]);
	if (e.target.classList.contains('fa-trash-can')) {
	listofTasks.deleteTask(index);
	printTask(listofTasks);
	listofTasks.saveLocalStorage();
	} else if (e.target.classList.contains('fa-square')) {
	listofTasks.tasks[index - 1].complete = true;
	printTask(listofTasks);
	listofTasks.saveLocalStorage();
	} else if (e.target.classList.contains('fa-square-check')) {
	listofTasks.tasks[index - 1].complete = false;
	printTask(listofTasks);
	listofTasks.saveLocalStorage();
	}
	});

• Include a specific function instead of writing it directly on the event listener.

to-do-list/src/modules/ui.js

Lines 53 to 54 in 659775d

	ulContainer.appendChild(taskContainer);
	taskContainer.onclick = (e) => eraseTask(e, index, task, complete);

• Instead to add a click event inside the print task function, could be better if you do it externally. It will help to avoid possible bugs when users click on the tasks.

Happy coding! 😃