peer-to-peer code reviews

[carlosmuerto]

peer-to-peer code reviews

nice work, please check this:

• [ ] update git links in the package.json to this repo links https://github.com/MrOmachi/To-do-list-App/blob/20ceeb0e07fb981bef6eb9549403ada16bf9eec0/package.json#L13 https://github.com/MrOmachi/To-do-list-App/blob/20ceeb0e07fb981bef6eb9549403ada16bf9eec0/package.json#L19

• [ ] rather than parentElement.parentElement.children[2].children[0] use a querySelector with id, this if the HTML structure change it will not brake the code. https://github.com/MrOmachi/To-do-list-App/blob/20ceeb0e07fb981bef6eb9549403ada16bf9eec0/src/index.js#L23-L42

• [ ] is good practice to name the file with the class it has (each file should have only one class), and the class name should be representative of the type of the object it constructs, in this case, should be toDo or `task https://github.com/MrOmachi/To-do-list-App/blob/20ceeb0e07fb981bef6eb9549403ada16bf9eec0/src/modules/todoConstruct.js#L1-L9

[Aseadd]

• It will be better to create DOM elements using Javascript instead of using plain html inside JS https://github.com/MrOmachi/To-do-list-App/blob/20ceeb0e07fb981bef6eb9549403ada16bf9eec0/src/modules/domDisplay.js#L20-L36
• Avoid DRY https://github.com/MrOmachi/To-do-list-App/blob/20ceeb0e07fb981bef6eb9549403ada16bf9eec0/src/modules/localStorage.js#L28

[Aseadd]

• Avoid DRY in your SCSS file https://github.com/MrOmachi/To-do-list-App/blob/20ceeb0e07fb981bef6eb9549403ada16bf9eec0/src/styles/main.scss#L60-L67