Check for JS Best Practice

[miltonHenschel]

• 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) {

• [ ]