Design comparison
Solution retrospective
Hello everyone. I had fun with this challenge. Would appreciate any feedback on how to improve the design. Thank you :)
Community feedback
- @Zy8712Posted 12 months ago
Your website looks great! The only slight modification I'd make is the bottom margins for the text of the "New" section when it is in its mobile format (right now the text is a bit squished). One other thing worth mentioning is that I feel like you should have <li></li> tags around your mobile menu icon:
<div class="hamburger-menu-close-container"> <img src="./assets/images/icon-menu-close.svg" alt="hamburger-close-icon"> </div>
Reason being is that screen readers have a specific way of announcing lists that only works if the list is properly structured. You can read more about it here.
Besides that your solution is very solid. Good job! 👍
1@Julie-GzPosted 11 months ago@Zy8712 Thank you so much Bryan. I agree, the page does look strange on some screens, especially tablets and Ipads. I'll take your advice and try to improve my solution. Thanks again :)
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