Design comparison
SolutionDesign
Community feedback
- @MrDevMPosted 12 months ago
Great job on your solution. I looks fantastic.first I noticed that the h1 font is too small give font size to h1's around 40px
h1{ font-size:40px; /* or 2.5 rem*/ }
secondly is give lien-height for paragraphs it look like close one to other so give a line-height to paragraphs
p{ Lien-heigh :25px; /* or 1.5 rem*/ }
Finally button part it looks 100% width instead give padding around 15px top bottom and 35px left right padding and a font-weight
button{ padding:15px 35px; font-weight:bold; }
Thanks.
Marked as helpful8@EmiLy-Ly-sanPosted 12 months ago@domxss Thanks for your review ! So many details to look at 😲. I need to improve more my work. It's really interesting to have your return for it 😊.
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