Design comparison
SolutionDesign
Community feedback
- @hitmorecodePosted over 1 year ago
Congratulations well done. Just a few things to fix some issues. If you add flexbox to the body, it will center the content in the middle of the page
body { font-family: 'League Spartan', sans-serif; background: url(images/bg-pattern-top-desktop.svg) no-repeat, url(images/bg-pattern-bottom-desktop.svg) no-repeat; background-position: -50px -50px, 0px 0px; min-height: 100vh; /* add this */ display: flex; /* add this */ justify-content: center; /* add this */ align-items: center; /* add this */ }
When using flexbox you don't need to add
flex-direction: row
. Flexbox is by default set to row.Marked as helpful1@ahmed75tgPosted over 1 year agoHi @hitmorecode , thanks for the tip, I'll try to apply it in my next projects, have a nice day.
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