search

DeVuDeveloper / To-Do-list

Add todo, delete todo and remove all completed
https://vudej.github.io/To-Do-list/
MIT License
6 stars 0 forks source link

Peer-To-Peer #11

Open tresorsawasawa opened 2 years ago

tresorsawasawa commented 2 years ago

Great job with your code :+1: :100: , it's great and clean. However, there is one thing you can improve to make your page work perfect. I think it will be better to make your return button work when the user wants to add a task. rtn

guerreiropedr0 commented 2 years ago

Hi Dejan, your code looks amazing! I see that in the render.js file, line 7 to 40, you used a loop, and every time you create elements and then append it. This is correct, but I think it can look better if you use template literals, this way the code will significantly be shorter and better

banlon-jones commented 2 years ago

Great job :tada:, @VuDej your code is DRY, simple, and easy to understand however I believe it could be great to refactor your render function into multiple functions this is [optional]

Once more great job :clap: :clap: