JavaScript in the browser

[enionsouza]

JavaScript in the browser

• [x] Do not commit console.log to your repo. It’s ugly, it kills performance and it can make confidential data be visible to anyone using the browser tools to look at your website.
• [x] Do not use window.alert() or window.confirm(). It’s ugly, impossible to style, it stops code execution and displays differently on different browsers. Use custom modal instead.
• [x] Keep the number of changes/updates to the DOM as low as possible, they are very expensive for the browser.
• [ ] Keep the application logic separated from DOM manipulation tasks.
• [x] Do not use document.write or eval
• [x] Add node_modules dir to you.gitignore file as all those files are not needed in your repo (each team member will install all packages thanks to your package.json file).

[notlfish]

https://github.com/enionsouza/TODO-List/blob/4e3cfa13904d5f3ab481681e2ab021b642ca79a8/src/todo.js#L21-L25 forEach allows you to take the index as an extra argument, like this:

    this.tasks.forEach((task, index) => {
      task.index = index;
    });