-
Notifications
You must be signed in to change notification settings - Fork 33
Open
Description
@kensteph
Hi Henschel, You did a great job!
HIGHLIGHTS
- Use of semantic HTML
- Use of ES6 features
- Clean code
SUGGESTIONS
- Please, consider adding the 'checked' property for every checkbox that has a completed todo even when the page reloaded.
- When editing a todo, if the description is empty, I think it would be better to remove this todo.
Happy coding!!!
- Please consider removing the "reload page" method when adding a new task to the list. it is an expensive operation memory wise and it is not strictly necessary.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels