Peer to Peer Code Review

[JohnFTitor]

Hello @memelopez.

Your project is looking great! 💯 💯 I particularly like the way you handled the event listeners taking advantage of the bubbling and the additional buttons to add or edit the tasks.

Suggestions 🚀 🚀

Here I have some suggestions that may improve your code.

• [ ] This function may be unnecessary since JavaScript can change the data type of a string. If it is empty, null, or undefined, it would be false and if not, it would be true. Following that, you can just check the word inside the if.

https://github.com/memelopez/toDoList-webpack-mv/blob/8c7528ede4f6fe9cbd6d31ba3d3faa4435b61ee0/src/index.js#L15

• [ ] Here, instead of calling the function that parses the entire array, it may be better to call the function that adds only one task to the DOM addTask2list(task)

https://github.com/memelopez/toDoList-webpack-mv/blob/8c7528ede4f6fe9cbd6d31ba3d3faa4435b61ee0/src/modules/ui.js#L106

Closing 😸

That's all from me. Feel free to contact me if you have any questions. Keep up the good work!

[zhadier]

Hey Elmer

Great job on the code, absolutely loved the structure of your Javascript and the overall design was 🔥

Suggestions

• Maybe adding an event listener to keypress will be a cool user-friendly feature