Design comparison
SolutionDesign
Community feedback
- @Mohammedabbas7Posted 10 months ago
Hello
Congrats on completing this challenge๐๐๐๐.
Nice solution and also like what you did with the attribution.
I noticed a few things in your solution you need to adjust:
- First instead of adding
height: 100vh
to the body element because this causes some overflow issue so addmin-height: 100vh
to make the responsive. - Second instead of adding
height: 60%
to the .content element just add gap propertygap: 1rem
. - you did add a height property to .container element that is not good practice, so don't add specific height to elements unless it's so important to add it.
I hope my advice is useful.
Happy Coding :)
0@AbuSalman143Posted 10 months ago@Mohammedabbas7 Thank you so much for the valuable advice given. In the future, I will make sure to apply your advice to further projects.
0 - First instead of adding
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