@ApplePieGiraffe
Posted
Hey, Abdihafid Adan! 👋
Good woprk on this challenge! 👏 You've done a great job at completing this challenge with vanilla JS! 🙌
A few things I'd like to suggest are,
- Adding
object-fit: cover
to the background image in the page so that the image does not seem stretched or squeezed when its aspect ratio/dimensions change. - Using more semantic HTML elements in your markup, which is important for accessibility and SEO-related reasons. For instance, the theme toggle and the options and the bottom of the to-do list could really benefit from being wrapped in
button
elements. That way, users can interact with those elements using their keyboard, which is important for users who do not have or use their mouse/trackpad. 😉
Hope you find these tips helpful. 😊
Keep coding (and happy coding, too)! 😁
Marked as helpful
@Abdul400
Posted
@ApplePieGiraffe Thank you so much for your feedback. I will most definitely consider them for my next project.
@ApplePieGiraffe
Posted
@Abdul400
Glad to help! 😀