@mattstuddert
Posted
Very nice work on this challenge, Remus! Your solution looks excellent, vs the design and the animations you've added look smooth and well thought out.
My primary piece of feedback would be to focus on building access interfaces. At the moment, you're using non-interactive elements to trigger events. This means anyone not using a mouse/trackpad to navigate can't interact with the app. Instead, use interactive elements, like a or button elements to trigger actions. Buttons are perfect for events like toggling the theme and the state of the todos, as you're staying on the same page. Anchor tags are best when you're expecting to navigate elsewhere.
I'd also recommend reviewing the responsiveness of the project. The layout looks great, but the todo list jumps to full-width pretty early, and the All/Active/Complete toggles at the bottom get bunched up.
I hope that helps. Keep up the incredible work! Were there any big lessons you learned while building this project?
Marked as helpful
@Miculino
Posted
@mattstuddert Thanks a lot for the constructive feedback!
I did want to use a / button in the beginning, but I probably got distracted and forgot about them altogether. As for the responsive side, thanks a lot for pointing those issues out too. I'll try to fix them asap.
In terms of big lessons, I learned how to have the frontend technologies communicate with the backend ones, implement some best practices that I've rarely used to write better, cleaner, and more structured code and work with MongoDB!
@mattstuddert
Posted
@Remus432, you're welcome! It sounds like you learned loads, which is awesome. Keep it up! 👍
