@pikapikamart
Posted
Hey, good work on this one. The layout in desktop and mobile is good.
Suggestion/s would be that
-
Scrollbar. Upon loading your site, there is a scrollbar at the bottom. This was caused by the declaration of
width: 100vw
in yourbody
tag. So fix this, simple remove that declaration. -
Font size. Right now, your font-sizes are bigger compared to the original, thus creating some sections a lot bigger. So please consider resizing them so that it will be really good to look at.
-
Mobile state. In your mobile state, your h1 size does not really scaled well so it is really big right now. Scaling it down will be awesome. Also there is an appearance of scrollbar at the bottom, again. As I can see, you have a padding in your
body
tag right. Then everydiv
inside one level deep have a padding again, which I think is not really necessary. Removing those padding of left and right will be awesome. Also removing themin-width
of those div, and you can change them to take the whole width of the container. Since there is a padding from thebody
it will make sure that your content will not be touching the sides of the viewport.
Overall, you did a great job^
@tin-pham
Posted
@pikamart Oh thank you so much ^ ^. Very appreciate, I will fix these stuff