IbrohimRasulov / To-Do-list

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Peer-to-Peer Code Review

codepantha opened this issue Β· comments

Hi @IbrohimRasulov πŸ‘‹ πŸ‘‹

You have a beautiful project.

Things to highlight: πŸ‘‡

  • Professional README.md βœ”οΈ
  • Clean UI βœ”οΈ
  • Drag and Drop functionality works really well βœ”οΈ

Suggestions to hopefully make your code better:

  • I noticed that you use spaces sparingly in your code. Any reason for this in particular? I believe if you have spacing in your code, you will be able to separate your thoughts more clearly and make your code more readable. What do you think?

Here's an example of what I'm talking about:

To-Do-list/src/index.js

Lines 59 to 68 in 63727c0

listIt();
document.querySelector('#taskForm').addEventListener('submit', (event) => {
event.preventDefault();
task.add(list);
listIt();
});
document.querySelector('.clearer').addEventListener('click', () => {
task.removeDone(list);
listIt();
});

Thanks for sharing your code

Cheers and happy coding πŸŽ‰ πŸŽ‰

Thanks @ codepantherr, I really appreciate your suggestions. I totally agree with what you said, I try to fix the issue!