Todo app

Community feedback

• 

  Matthias•630

  @astragenius

  Posted over 3 years ago

  Hey your bg-images and icons dont load correctly.

  Remove "/" on all your img url path.

  for example:

  body {
      --bg-img: url("/images/bg-mobile-dark.jpg");
  }
  
  to
  
  body {
  
      --bg-img: url("images/bg-mobile-dark.jpg");
  }
  
  

  do this on all your image urls. After this all images should load correctly.

  01

  Marked as helpful
• 

  Progress Aienobe•565

  @Paienobe

  Posted over 3 years ago

  To do the deleting, you could push all the items added to the list to an array and then when you want to delete something, you can use filter method and return only the items not equal to the item you want to delete. Hope this helps

  01

  Marked as helpful
• 

  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 :)

  00