Design comparison
Community feedback
- @kush-x7Posted almost 2 years ago
Hello Alice Auko Owiti, You have done good work! 😁
I have analyzed your code and found the bug which is causing you an overflow issue
- You have given
width:1440px
in the universal selector which is causing you an overflow issue, just remove it, and then your site will look perfect
Keep learning how to code with your unique solutions to challenges.
Hope this help 😉 and Happy coding!
Marked as helpful1@aliceaukoPosted almost 2 years ago@kush-x7 Thank you. I've done that and everything is okay now. Onto the next.
0@kush-x7Posted almost 2 years ago@aliceauko I rechecked your code and found one more bug.
-
Remove
margin-left: 570px;
margin-top: 200px;
from your container because when you try to resize or maybe open your site on smaller devices than it will give an overflow issue. -
Add the below code in your main element and it will center your div
display: flex; justify-content: center; align-items: center; height: 100vh;
Marked as helpful1@aliceaukoPosted almost 2 years ago@kush-x7 Thank you so much. I've learnt a lot.
0@kush-x7Posted almost 2 years ago@aliceauko I am here to help, so whenever you have a doubt just tag me. Don't know whether the tag function work or not. If it's work then surely I will review your code.
One thing more just removes
background-color: greenyellow;
from.img-contain
, because if the image take more time to load then the bg color is showing , which is not looking goodMarked as helpful1@aliceaukoPosted almost 2 years ago@kush-x7 Sure I'll be tagging you. Let me remove that. Thank you
0 - You have given
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