Adriano• 34,090
@AdrianoEscarabote
Posted
Hello Deepak, how are you? Welcome to the community of front-end mentors! I truly loved your project's outcome, however I have some advice that I hope you'll find useful:
A good practice to center content is using grid or flex-box, avoid using margin or padding to make placements, use only in the latter case! we can do it like this:
Flex-box:
body {
display: flex;
align-items: center;
justify-content: center;
flex-direction: column;
min height: 100vh;
}
GRID
body {
display: grid;
min height: 100vh;
place-content: center;
}
To improve the structure of the html document, prefer to wrap all the content with the main tag, since this challenge is just a component, it needs a main tag to know what the main content of the page is!
The remainder is excellent.
I hope it's useful. 👍
Marked as helpful
0
