Skip to content
Submitted about 2 months ago

Todo app

Jennifer Acosta•1,200
@jennawlin3
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Todo app
  • HTML
  • CSS
  • JS
3intermediate

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

Well, I learnt a lot from this project, i'm happy of the results, there are some things I want to improve but i'm happy with this project

What challenges did you encounter, and how did you overcome them?

I had to fix some bugs, I get stuck a couple days with some bugs, but It works

Community feedback

  • Ilya Andreev•750
    @NeoScripterPosted about 2 months ago

    Hey, I viewed your solution and would like to give a few suggestions for improvement: 1) The background image does not change while switching the theme. You could leverage CSS custom properties to specify a path to the other image when the theme changes. Also, I think it would be better to make the background-size contain instead of fit, as I can see only the top of your bg image now; 2) The font color for todos in dark mode is too pale, I can barely see it. 3) When I drag and drop in the middle of the rows, flex-direction becomes row for some reason instead of column. Basically, the todos start stacking in a row instead of a column. You could look into todo box parent CSS properties to find the reason. 4) I would add adding a todo with an enter button for convenience. I hope these suggestions help!

    Marked as helpful
    1
    Jennifer Acosta•1,200
    @jennawlin3Posted about 2 months ago

    @NeoScripter thank you for your feedback, I will check that

    1

Please log in to post a comment

Log in with GitHub

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