Design comparison
SolutionDesign
Solution retrospective
Please take a look and give me some feedback. Thanks
Community feedback
- @correlucasPosted about 2 years ago
πΎHello Omar, congratulations for your new solution!
Great solution here, clean code, amazing design. The only thing you've missed is alignment, I'll give you some tips to fix it:
All you miss is to give the content the proper vertical alignment with flex properties like
display: flex; align-items: center; flex-direction: column; justify-content: center;
and addmin-height: 100vh
to make the child element align vertically:body { min-height: 100vh; color: #fff; font-size: var(--main-fs); font-weight: 400; font-family: var(--main-font); padding: 2.1875rem 1.875rem; background-color: var(--bg-body); background-image: url(/images/bg-mobile.svg); background-repeat: no-repeat; background-position: top; background-attachment: fixed; display: flex; align-items: center; flex-direction: column; justify-content: center; }
π I hope this helps you and happy coding!
Marked as helpful2@omarsaleh11Posted about 2 years ago@correlucas ty, you are amazing, you helped me a lot
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