Submitted over 3 years ago

Mobile first using CSS variables and media queries

Lilibell• 60

@Lilibell

Preview site View code

This is a solution for...

Todo app

HTML
CSS
JS

3intermediate

View challenge

Design comparison

SolutionDesign

Solution retrospective

Hey everyone,

this is my first Intermediate Level challenge and it took me forever to complete. However it was a lot of fun for me and I've learned tons of new things. I celebrated everything that finally worked as I wanted it to! Although by now everything seems to work, my code feels a little messy to me. I would be very glad if someone takes the time to review the structure of my code and probably gives me some tips on how to improve its readability and if possible make it less redundant. Of course I am happy for any other feedback as well!

Edit: according to the preview, there also seems to be an issue with my sort buttons, but I don't see it when I open the page... any help would be welcome! Thank you for your time.

Community feedback

Yadhavan Prabhakar• 460

@yadprab

Posted over 3 years ago

Hey, you did a great job. I looked at your code just a few things to change. Change the overflow-scroll to auto and don't set a fixed height for the #todo instead put them in a container set a max-width. My suggestion is you don't need to show the todo area generate that area via js according to user inputs. I'm happy that you Took the intermediate challenge. And for more interesting stuff learn about insert-adjacent Html and local storage it will more fun trust me.

Keep learning All the best

Please log in to post a comment

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