Todo list app using vanilla js , css flexbox

Community feedback

• 

  Jeuri Morel•1,385

  @JeuriMorelPosted over 2 years ago

  Hey Divyank, nice job.
  I see you're saving the todos to localStorage as an array with just the todos. Maybe you can also add a property for if it's completed or not. So instead of

  ['eat', 'go for a walk']
  

  you can have something like this:

  [{todo: 'eat',
  completed: false},
  {todo: 'go for a walk',
  completed: true}]
  

  I also noticed that on large screens your background image doesn't stretch all the way across. A background-size: contain; should do the trick on large screens, so maybe you can add that to a media-query.

  Marked as helpful

  0

  ---

  

  Divyank Dubey•60

  @dubeydivyankPosted over 2 years ago

  @JeuriMorel Thanks Jeuri for the suggestions. I'll try to do it the way you mentioned and thanks for pointing out the issue about the background image as well.

  0