Todo App with HTML, CSS & JS

Community feedback

• 

  P

  Sara Dunlop•430

  @RiscloverPosted 4 months ago

  You could do something like this. It basically takes the trimmed (trimmed meaning spaces are ignored) input value's length. The length being 0 means that the input box is empty, so if it isn't 0, allow the todoDiv element to be appended to the list element.

  if (todoInputs.value.trim().length !== 0) {
      list.appendChild(todoDiv);
  }
  

  If you have any further questions or if I misunderstood your original question, feel free to let me know!

  Marked as helpful

  2

  ---

  

  Dugidem•160

  @DeeNue36Posted 4 months ago

  @Risclover Thank you. Will give this a try immediately and let you know how that turns out

  0

  

  Dugidem•160

  @DeeNue36Posted 4 months ago

  @Risclover Works like magic. Thank you so much.

  0

  

  P

  Sara Dunlop•430

  @RiscloverPosted 3 months ago

  @DeeNue36 I'm very late with this response, but yay! I'm glad it worked out. No problem.

  1
• 

  Rahmonbek•370

  @Rahmonbek-0001Posted 4 months ago

  you have to work a lot with this code

  0

  ---

  

  Dugidem•160

  @DeeNue36Posted 4 months ago

  @Rahmonbek-0001 I don't understand. Do you mean I have a lot to work on with this code? If that's the case, like I mentioned, it's a really old project, about three years now when I started programming full time. I forgot to submit it and a couple of other challenges

  1

  

  Rahmonbek•370

  @Rahmonbek-0001Posted 4 months ago

  @DeeNue36 sorry i didn't know that

  0

  

  Dugidem•160

  @DeeNue36Posted 4 months ago

  @Rahmonbek-0001 It's alright, criticism is welcome. Will definitely retry the challenge should time permit.

  0

  

  P

  Sara Dunlop•430

  @RiscloverPosted 3 months ago

  @Rahmonbek-0001 I know you guys already resolved this, but I wanted to add that I think it would be a lot more constructive if, in the future, you gave the person specific examples instead of just simply telling them that their code needs work. :)

  2