Hi Edahi, here's a summary of our code review session:
Overall looks good - good use of naming and JS modules
To DRY taskList, extract localStorage.setItem into a new function
As DOM operations are expensive, instead reloading the full task list, please try to replace and remove elements from the list instead of reloading the full list
From Jose;
instead of repeating the check elements, you could make this conditional if checked is true and using the same code for the check elements
instead of using if(x === true) {...}, replace if(x) {...}
Separate class actions from functions for DOM operations
Hi Edahi, here's a summary of our code review session: