Hey @Sonikak004 , good job implementing the features of the To-Do List project! 👍

Highlights

Clean code 👍
Minimal design based on template 👍

-There is one small issue I would suggest to prevent from adding empty task in your project.

Highlights 🎇

• Did not commit console.log to the repo
• Added node_modules dir to the .gitignore file
• The code is organized into functions
• Arrow functions (() => {}) are used to define functions

The javascript file adheres to several common best practices, but there may be additional considerations.

I see the whole code is very minimized and I appreciate this 👍 You have done a great job 🥇

The code is modularized, but I think you have a better chance to make it more modularized by outsourcing some code from index.js files. I suggest you check and optimize more.

I love how you coded in a very minimized fashion. Great work 👍

- DRY: Don't repeat yourself!

• To-Do-list/src/index.js

  Line 36 in 2db1955

  const taskDescription = document.createElement('input');

  
  - for the above function you have repeated task description too many times, try minimizing it.

- KISS: Keep It Simple, Stupid.

• The code takes a direct approach to accomplish the tasks

- YAGNI: You Aren't Gonna Need It.

• There are no signs of extraneous or unused code
• It doesn't include additional functionality

Hey @Sonikak004 👋🏻

You have done a great job so far. 👍🏻 But there are still some issues you have to fix them. ♻️

Highlights :
✅ Good UI
✅ Clean code
✅ Good implementation of requirements

• Try to remove disabling of eslint as it is not a good coding practice.

Things to address : ♻️

• To-Do-list/src/index.html

  Line 12 in 2db1955

  <div class="container">

  • Instead of using div's like above you can use semantic tags like header,nav .

• To-Do-list/src/style.css

  Line 117 in 2db1955

  font-family: 'Inter', sans-serif;

  • Instead of repeating the font family, create a class with the font and apply the class wherever needed.

Highlights 🎇

• Did not use inline styles
• used the class attribute for multiple elements
• Close all HTML elements.