
Design comparison
Solution retrospective
Please provide feedback
Community feedback
- @ApplePieGiraffePosted about 4 years ago
Greetings, wenadev! 👋
Nice job on yet another challenge! 👏 Your solution looks great and responds nicely and I quite like the hover states you added to the social media icons in the footer of the page! 👍
I'd like to suggest,
- Adding a
max-width
to the content of the page to prevent it from becoming too wide on extra-large screens (I personally like to wrap the content of each section of the page in adiv
calledcontent-wrapper
or something and add the stylesmax-width: 1440px
andmargin: 0 auto
to thatdiv
to keep the content centered on larger screens). 😉 - Not adding a hover state to the paragraph below the heading in the hero section of the page (since it isn't a clickable element and hovering over it makes the text disappear).
Of course, as always, keep coding (and happy coding, too)! 😁
0 - Adding a
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