Design comparison
Solution retrospective
my code was simple.
Community feedback
- @olalemiPosted almost 3 years ago
-
body{ background-image: url(images/pattern-background-desktop.svg); background-repeat: no-repeat; background-size: in pixels; height: 100vh; background-color: #E0E8FF;
-
For responsiveness lookup media query via https://www.w3schools.com/css/css3_mediaqueries.asp
-
your margin for your content should be defined margin-top: what is applicable px;
Hope this helps
0@FORTUNE21Posted almost 3 years ago@olalemi Thank you for providing those useful tips. I have done the needful. All fit well now.
0 -
- @olalemiPosted almost 3 years ago
Nice work, but I think you can improve on these areas
- Your Final design was not responsive.
- Your background was disproportionate, I looked at your code, your background size should not be cover
- You didn't give room for your margin-top, it's hitting the top of my screen
Hope this helps
0@FORTUNE21Posted almost 3 years ago@olalemi Thank you for your feedback. Can you help provide the appropriate code snippet to use for the stated areas to improve on.
Thank you!
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