
Design comparison
Community feedback
- @4-WalkerPosted 14 days ago
Nice work! I enjoy how you set up the :root styling to allow flexible color changing throughout the website!
Most of the things that could be improved that I noticed when looking at your code relates to usability. For example - images should be provided 'alt' text to help with screen readers. Also, some containers should be provided with roles as well. Your 'container' class div should be given a 'role=main' property as it holds the main content on your webpage. This is also helpful for people navigating websites with screen readers. Good luck with future challenges!
Marked as helpful0@Shahd-AbdalghnyPosted 14 days ago@4-Walker Thank you so much for your feedback! I'm really glad you liked the way I set up the :root styling. I truly appreciate your suggestions regarding usability. You're absolutely right about adding alt text for images and assigning appropriate roles to containers—I’ll make sure to implement these improvements. Thanks again for your valuable insights, and I look forward to improving my work in future challenges!"
0
Please log in to post a comment
Log in with GitHubJoin 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