-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Hi @Mesi21 👋,
You have done an amazing job. 💪💪
I love your UI. 💯 It is so smooth.
Suggested Changes ♻️
However, there are a few suggested changes you should consider making
- Rather than using the
window.location.reload();, consider calling thedisplayTasks()functions and clear the list before looping to avoid list duplication.
Line 49 in 9013000
window.location.reload();
Line 65 in 9013000
window.location.reload();
Line 77 in 9013000
window.location.reload(); - Rather than use
add.appendChild(newBtn);a couple of times on the same node element, consider usingappendwhich allows appending multiple children.
ToDo/src/elements/createTaskArea.js
Lines 14 to 17 in 9013000
add.appendChild(addNewTodo); add.appendChild(newBtn); taskArea.appendChild(header); taskArea.appendChild(add);
Happy hacking 😏😏😏
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels