Design comparison
SolutionDesign
Solution retrospective
Any feedback would be much appreaciated!
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hello, once again, Sandro! 👋
Nice work on another challenge! 🙌
A few small things I'd like to suggest are,
- Adding a little more space between the tops/bottoms of the company logos in the mobile layout.
- Making sure there's always a little bit of space between the logo and the navigation links in the desktop layout (probably by adding some margin/padding between them).
- Making sure the
sr-only
<span>
tags next to the social media icons in the footer of the page don't show up (they seem to be visible, at the moment).
Hope those tips help. 🙂
Of course—keep coding (and happy coding, too)! 😁
1@SanidethPosted over 3 years ago@ApplePieGiraffe Hey, thanks again, will fix those shortly
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