@pikapikamart
Posted
Hey, great work on this one. The layout in both desktop and mobile view is good and that custom scrollbar really suits well. Layout resizes well when going in mobile view.
My only suggestion would be that.
I think it will be better if you set the padding
in the .block
selector, not relative to the viewport's height. Since right now, if I inspect your layout in dev tools, since your padding is relative to the height, hence vh
. It scales a lot, making your layout seems getting squished in vertical axis. It would be better that the padding are fixed paddings, using rem, px or some sort. Also to make the main
selector using min-height: 100vh
instead of min-height: 70vh
.
Overall, your work is really good and the hovers states as well^^
@emiomacollins
Posted
@pikamart thank you, I made the changes for the section padding, I really don't know what I was thinking :) , because on landscape mode in phones the sections actually are squished cus the vh is small. thanks for the tip. Also, I purposely set the desktop hero section to be 70vh. I just personally thought it looked better.