Skip to content
  • Unlock Pro
  • Log in with GitHub
Solution
Submitted over 4 years ago

To do App

Val•95
@el-shredders
A solution to the Todo app challenge
View live sitePreview (opens in new tab)View codeCode (opens in new tab)

Solution retrospective


Hi all! Here is my take on the todo app challenge. done in html/css and vanilla javascript. It's still not 100% finished:

  • i haven't implemented the drag and drop features (its a bit out of my league for now maybe)
  • I still have to do the "Clear Completed" feature.

I'd still have reviews or advices from you guys on a few points.

i m not sure I used the best Html structures to do this, for structure and semantics,

it was my first time adding a theme switcher, i used a single data attribute in my body to get it done, if you have any suggestions to improve it

I did the Js part in procedural programming, so it's a bit hard to keep everything clean (i tried my best and added lots of comments) So if you guys have any advice for me to keep my code cleaner or anything, i'll happily take it.

And i was wondering if it was a good practice to kill the event listeners on a todo Element removal, if so what would be the best practice to do it?

Thanks in advance Val

Code
Loading...

Please log in to post a comment

Log in with GitHub

Community feedback

No feedback yet. Be the first to give feedback on Val's solution.

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord

Stay up to datewith new challenges, featured solutions, selected articles, and our latest news

Frontend Mentor

  • Unlock Pro
  • Contact us
  • FAQs
  • Become a partner
  • Use cases

Explore

  • Learning paths
  • Challenges
  • Solutions
  • Articles

Community

  • Discord
  • Guidelines

For companies

  • Hire developers
  • Train developers
© Frontend Mentor 2019 - 2025
  • Terms
  • Cookie Policy
  • Privacy Policy
  • License