Cluab / to-do-list

"A "To-do list" is a tool that helps to organize your day. It simply lists the things that you need to do and allows you to add, edit, delete, and mark them as complete. Built with Es6 and Webpack and added some test cases using jest as its framework"

JavaScript best practices

Cluab opened this issue 2 years ago · comments

Ebrahim Suhail Al-Yousefi commented 2 years ago

instead of using the same border multiple times, I can use a class attribute

https://github.com/Cluab/to-do-list/blob/master/src/style.css#L36

add all of the global consts to the start of the page.

https://github.com/Cluab/to-do-list/blob/master/src/index.js#L7:~:text=from%20local%20storage-,const%20retrieveData%20%3D%20localStorage.getItem(%27list%27)%3B,const%20body%20%3D%20document.querySelector(%27body%27)%3B,-body.onload