@pikapikamart
Posted
Hey, great work on this grid challenge. First, the layout seems resizes well when going in mobile view, and the desktop view is good as well.
A couple of suggestions;
-
Overflow. Right now, your
body
tag have aoverflow: hidden
declaration on it. Now this prevents the user from scrolling down in desktop view, only for when opening dev tools. Removing that will be awesome. -
Paddings. Well this is just preference, but I think it will be better if your main component have a larger gap between the border of the screen, like the one on the original.
-
Line-height. Also adjusting the
line-height
so that the words will be more easy to read.
Well, those are just preferences of mine, but if you were to consider it, thank you for that.
Overall, you did really good job in here^^
@under4
Posted
@pikamart Hey thank you for the feedback, I'm glad you liked it. I totaly forgot the line height lol!!