3kori• 80
@3kori
Posted
Hi @MARCUS-IMPERATOR,
Congratulations on completing this project. It looks amazing!
I'd like to offer some suggestions to improve the solution:
- Set these global styles to avoid spacing and layout issues caused by browser default styles.
* {
box-sizing: border-box;
margin: 0;
padding: 0;
}
- Great job on using flexbox to center the container. Add
min-height: 100vh;
to the body to help with the centering. - To avoid squishing the container when changing screen sizes, consider using a high percentage for the container's width.
media screen and (width: 750px)
.container {
width: 95%;
}
- Finally setting padding for the body isn't necessary.
I hope you find this helpful!
Marked as helpful
1
MARCUS-IMPERATOR• 90
@MARCUS-IMPERATOR
Posted
@3kori Thank you so much, I will incorporate these changes right now.
0