Pros

• The code is clean;

Cons

• Remove function console.log() from the code;
  -Add to your dist folder into the .gitignore file

The app looks good and is working fine.

• Consider disabling edit mode after the todo is marked as completed
• Add an event listener to update local storage when marking todo as completed

• Keep all the JavaScript code in JS files, not in the script tag. 👍

• Keep your code clean

• ToDoList-App/src/modules/localStorage.js

  Line 5 in d68855b

  if (localStorage.getItem('todolists') === null) {

• ToDoList-App/src/modules/localStorage.js

  Line 22 in d68855b

  if (item.desc === newdesc) {

• ToDoList-App/src/modules/localStorage.js

  Line 42 in d68855b

  for (let i = 0; i < todolists.length; i += 1) {

• ToDoList-App/src/modules/localStorage.js

  Line 52 in d68855b

  for (let i = 0; i < todolists.length; i += 1) {

• ToDoList-App/src/modules/userInterface.js

  Line 21 in d68855b

  if (elt.classList.contains('text-btn')) {

• ToDoList-App/src/modules/userInterface.js

  Line 48 in d68855b

  // static compToDoList() {

• If you use node modules, .lock files should be in the repository to avoid problems with future versions of the dependencies. 👎

• If you use ES6, use object destructuring to get the values from an object. This way you can avoid repeating a lot of code. 👎

• ToDoList-App/src/modules/toDoList.js

  Line 2 in d68855b

  constructor(index, desc, comp) {

• [ ]

@kensteph
Hi Henschel, You did a great job!

HIGHLIGHTS

• Use of semantic HTML
• Use of ES6 features
• Clean code

SUGGESTIONS

• Please, consider adding the 'checked' property for every checkbox that has a completed todo even when the page reloaded.
• When editing a todo, if the description is empty, I think it would be better to remove this todo.

Happy coding!!!

@darkxdev

• Please consider removing the "reload page" method when adding a new task to the list. it is an expensive operation memory wise and it is not strictly necessary.