Abdul Khaliq 🚀• 72,660
@0xabdulkhaliq
Posted
Hello there 👋. Congratulations on successfully completing the challenge! 🎉
- I have other recommendations regarding your code that I believe will be of great interest to you.
CSS 🎨:
- Looks like the component has been centered properly. But the issue is the content inside the
body
causes overflow on smaller screens and let people scroll the content, this issue want to be cleared.
- This is just caused by over usage of
margin
property, we can usemargin
if necessary but for desktop version there's no need to use it
- Now set
margin
for thebody
in desktop mode, after setting you can able to see the changes
@media (min-width: 1130px) {
body {
margin: 0;
}
}
- Now the overflow of content has been cleared
.
I hope you find this helpful 😄 Above all, the solution you submitted is great !
Happy coding!
Marked as helpful
0
Chanawin Kamolpanus• 350
@Chanawin-kmpn
Posted
@0xAbdulKhalid Thanks for the feedback, it helped me to understand the body overflow in mobile mode.
1