imNotAli

Latest comments

• 

  Duyen Nguyen•950

  @Duyen-codes

  Submitted over 3 years ago

  

  Todo app

  3

  10
  

  imNotAli•230

  @AliNaderii

  Posted over 3 years ago

  Hey there :) Well, I tested your code and this is what I found.

  • Your querySelectorAll is not working as it should and when I used getElementsByTagName it worked properly.
  • The way that you are using spread syntax is not needed, querySelectorAll returns an array by default.
  • What you are pushing to todoItems is not necessary, you could just use querySelectorAll.
  • I think it's better to use createElement and then appendChild instead of insertAdjacentHTML.

  I hope you find it helpful :)

  0