Responsive clipboard landing page using html and css
Design comparison
Solution retrospective
Hello, everyone! š
Open to any feedback and recommendations, your suggestions matter a lot.
Happy coding :)
Community feedback
- @dwhensonPosted about 3 years ago
Hey @muhammadshajjar nice job on this one! š the page looks great and responds well at small viewports too! š Here's a suggestion or two you might like to consider:
-
The background image on the header looks a bit odd on large screens. Luckily, there's a few easy fixes for this. You could try adding
background-size: cover
on the header and this should fix things. This is a handy property and worth reading up on. -
At the moment, the content if your footer spreads out over the full screen width. I think adding your
container
class to the footer would help with this. Because, you need to keep the background color on the full width footer itself you might want to do this on adiv
element that wraps all the footer content.
Otherwise I think you've done a lovely job here! Great work!!
Cheers š
Dave
Marked as helpful2@muhammadshajjarPosted about 3 years ago@dwhenson Lovely suggestion from you, I have taken account of them and fixed them.
Thanks, Dave.
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