Design comparison
SolutionDesign
Solution retrospective
All feedbacks is welcome.
Community feedback
- @correlucasPosted about 2 years ago
๐พHello OLUWATOSIN, congratulations for your new solution!
Here's some tips for you:
Replace the h2 with a h1 since this is the main heading. I saw that you've used
br
to break paragraph line withmax-width: 320px
andmargin: 0 auto
to force the alignment, this is a better choice because this make the content fit the container and doesn't change just a single line like thebr
.To fix the background instead use
background-size: contain
.Hope this helps, happy coding!
Marked as helpful0@Oluwatosin0Posted about 2 years ago@correlucas Thank you for the tips
1
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