Todo-List-App icon indicating copy to clipboard operation
Todo-List-App copied to clipboard

Do I follow JavaScript best practices?

Open musangamfure opened this issue 1 year ago • 0 comments

According to JavaScript Best Practices, DRY, KISS, YAGNI, and HTML & CSS:

Here are some issues related to JavaScript best practices:

  • [ ] The code has a mix of arrow functions and traditional functions. It is recommended to be consistent with the use of arrow functions throughout the code.
  • [ ] Redundant code: The saveTodoItems() function is called twice in the addTodo() function.
  • [ ] Unused imported module: The checkboxCheck module is imported but not used anywhere in the code
  • [ ] Code repetition: The toggleDeleteIcon function is defined twice in the code, once as an imported module and once within the same file.
  • [ ] Large functions: Some functions, such as displayTodoItems() and addTodo(), are quite large and could be refactored into smaller, more modular functions

Here are the issues related to HTML and CSS best practices:

  • [ ] Lack of semantic HTML: The code uses div and li elements without any semantic meaning, which can make the code harder to understand and maintain

musangamfure avatar May 09 '23 14:05 musangamfure