Design comparison
SolutionDesign
Solution retrospective
Hello!
My first time using CSS Grid, but I think I got it to work at least for this site: P
I made a slight change in the menu, which I liked a little more.
Community feedback
- @tedikoPosted over 3 years ago
Hello, Jesus! 👋
Well done on this challenge! Your solution responds well. What I can suggest is:
- Add
aria-label
s to your.social-media
anchors to help accessibility. - Maybe give
max-width
to.header
so it doesn't stretch on bigger resolutions. You did that on.main
container. - Your
.button-menu
should be a<button>
element since it will trigger some action.
Good luck with that, have fun coding! 💪
0@jesusleonel10Posted over 3 years ago@tediko Thank you for your comments, suggestions applied 💪
0 - Add
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