abdessalem1998 / -to-do-list Goto Github PK

Issues found :

1. #8
2. #6
3. #7

Hello Abdessalam nice project so far the code is clean there are no errors, however, I would like to suggest some changes to improve your code.

Suggested changes ♻️

When I press enter it does not do anything, it would be better if you added an event listener on the keypress enter so people can input
without touching the mouse.
it would look better if you edited the background on the hover and set the buttons' cursor to the pointer so people know where to click.

Thanks 👍

Hey, there, good job setting up the tests and making them work! Here are some details you could work on to perfect your code!

• Great that you used ES6 syntax, however, it seems like you missed the spot! You could change this line to make all the code similar!

Good job on your project! Here's the feedback you could consider before moving on!

@tmampa:

• Very clean code and great use of arrow functions.
• Use of comments in code can look unprofessional but great for learning.
• Overall the project looks nice

@Azonkeu:

Great job! Keep up the good work.

@NataliaPoletaeva:

• Good work with declaring functions as constants!
• Think about reworking the sorting function, as the nested for loops are not the best decision to implement.

Happy coding!

Issues found in my code

• use code and functions that are reusable.
• Use for each instead of for loop

@BonkeGcobo
Well done on meeting the project requirements 💯 ,
Your project is well documented, and I love what you did styling it with CSS big ups 🚀

@kingqabil

Issues found in my code

• I have to name functions in more descriptive names.
• Initiate all variables at the top of the code file.

Issues Found in my code:

• use more descriptive naming for the styling classes.
• group styles that are common.