todo-app-using-html-css-bootstrap-js

Community feedback

• 

  Sami Maachi•230

  @Sami-21Posted about 3 years ago

  great job buddy most of the functionalities work perfectly. however, the design isn't optimized for large screens (1440px>) and small screens (550px<) . the theme transition is pretty decent but the border of the input field if flashing on theme switch. over all you did great managing the input ,keep up the good work.

  Marked as helpful

  1

  ---

  

  jeyaprakash•255

  @log-baitPosted about 3 years ago

  @Sami-21 thanks buddy.... i will correct the errors

  0
• 

  Folarin Akinloye•1,240

  @folathecoderPosted about 3 years ago

  Hi Pal! 😎

  Congrats on attempting another cool challenge. Kudos!!

  There are some important features you forgot to implement:

  1.) Drag and Drop functionality => A user should be able to re-arrange the list. 2.) localStorage => You did not store the current state of the app locally, this is why the inputs and theme settings clears when the app is refreshed. 3.) Theme Switch Implementation => You can easily make use of CSS to perform the theme switch by defining all the color variables under a class name and simply toggling the class on and off, instead of writing your code in JavaScript. 4.) The "TODO" logo can be implemented using letter-spacing in css. You don't need to literally space the text like you did on line 22 of your index.html => "T O D O"

  Overall, good job! 👌 Keep it up!

  Marked as helpful

  1

  ---

  

  jeyaprakash•255

  @log-baitPosted about 3 years ago

  @folathecoder thanks pal.... i understand these points buddy but im a absolute beginner so it was little hard to understand the drag and drop api and css variables i can simply copy and paste it but i dont want to do it so i will correct these points asap when i fully understand it anyway thanks man for your valueable feedbacks it boosts me

  0