Design comparison
Community feedback
- Account deleted
Hi @vonaser,
Nice work on your project, you have 4 html errors. These can be resolved getting rid of the <a> element and using cursor: pointer; in your button classes. That will remove three of them and the last error is because you use "" instead of "/" in your suv icon file path.
To check for errors before submitting you can use the html and css validator sites.
Also the reason you are having to use margin to center your main is because body is not a flex container. if you make body a flex container and use justify-content and align-items your main will be centered nicely.
Marked as helpful2@vonaserPosted almost 3 years agoHello @madzi-hove,
Thanks for the feedback. The idea that making body a flex helped me a lot. I updated it according to your sayings.
0Account deletedHi @vonaser
And also try give your body a height: 100vh;. If you look at your page in your web developer tools and you hover your arrow over body in the inspector window, you will see that body does not come all the way down to the bottom, i think that is why your card is so high up. If you do not know how to use your browser developer tools it is very important you learn.
Marked as helpful1
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