Design comparison
SolutionDesign
Community feedback
- @lynaIFRPosted 4 months ago
Hi, congrats on completing this project! It looks good.
I took a look at your code, it's well written, but here are some things you can do better:
- Always use
rem
forfont-size
neverpx
. - To make a good CSS reset, I invite you to check Josh Comeau CSS Reset, It was very useful for me.
- You should set a
min-height : 100vh;
for the body, so that it can use all the frame height. - In the HTML, I think it would be better to contain the elements in a
div
or more, because using values for themargin
andpadding
to center elements is a bad practice. Check out Anamol Soman article to have a better understanding on how to do it.
I hope this feedback will be helpful for you in the upcoming challenges, keep it up 😁
Marked as helpful0@jonaschaconPosted 4 months agoThank you for the compliments and the valuable feedback!😃@lynaIFR
0 - Always use
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