P
visualdennis• 8,295
@visualdenniss
Posted
You have done great! It looks very nice on a desktop view.
My suggestion would be add min-height of 100vh (making this equal to full viewport height) instead of using 600px height for the container and remove the margin from it. This way you can get rid of the extra white space you have around the page. E.g.:
.container { display: flex; width: 1440px; background-color: hsl(220, 15%, 55%); box-shadow: 2px 7px 15px rgb(163 163 163); justify-content: center; align-items: center; min-height: 100vh; }
Keep up the good work and good luck on your journey.
Hope this feedback was helpful!
1
Chimatara Kalu• 70
@Chimatara
Posted
@visualdenniss Thank you so much for the feedback, I'll implement it ASAP.
0
