Luka Glonti• 3,440
@lack21
Posted
Good job 👍, but I have some recommendations!
- Remove
width: 100vw
from thebody
,body
is a block level element which by default takes full width available, sowidth: 100vw
does nothing here! - Replace
height: 100vh
tomin-height: 100vh
in thebody
, the difference is that, when you setheight: 100vh
to something, that means it won't be bigger than that, it might cause some problems in the future, so better approach is to setmin-height: 100vh
, like this in case content is overflowing container will adjust to it, after that you can as well removeheight: auto
from thebody
!
Marked as helpful
0