Morning session: Peer-to-peer code review 01/31/23
jorgeabrahan opened this issue · 0 comments
jorgeabrahan commented
Great Job so far!
Highlights
✅ Nice Commits
✅ Use of JavaScript/ES6 modules
✅ App is working as expected
Suggestions
Even though the app is working perfectly fine, there are just a few suggestions:
- As a suggestion (optional) whenever the user edits a todo and leaves it blank, in order to follow the guidelines of the functionality shown here: web.archive.org you should remove it.
- Also when the user is editing the todo and is checked, you can remove the line-through so that is easier for the user to read its content.