Abdul Khaliq 🚀• 72,660
@0xabdulkhaliq
Posted
Hello there 👋. Congratulations on successfully completing the challenge! 🎉
- I have some recommendations regarding your code that I believe will be of great interest to you.
CSS 🎨:
- Looks like the component has not been centered properly. So let me explain, How you can easily center the component by just adding an explicit width for your site.
- You already using
Flexbox
for layout, but you didn't utilized it's full potential. Just add the following rules to properly center the component.
body {
min-height: 100vh;
}
- And apart from this, i have noticed that you used single-hand properties for setting background. So i want to share you the snippet for short-hand property.
body {
background: top/contain url(./images/pattern-background-desktop.svg) no-repeat, #E1E9FF;
}
- Now your component has been properly centered along with setting the background using short hand property.
.
I hope you find this helpful 😄 Above all, the solution you submitted is great !
Happy coding!
Marked as helpful
0
Tatyane Gonçalves• 360
@tatyanepgoncalves
Posted
Hello, @0xabdulkhalid!
Thanks for the feedback!
0