@pikapikamart
Posted
Hey, awesome work on this one. Layout in desktop looks good, though it is smaller, the mobile layout looks fine but some sections have big white-spaces and like what Fluffy Kas said, a bug occurs.
Some suggestions would be:
- Always have a
main
element that will wrap the main content of the page. The.container
selector could usemain
element instead ofdiv
. - Avoid using
height: 100vh
on a large container like the.container
selector. This makes the element's height only limited to the remaining viewport/screen's height. Remove this or replace it withmin-height: 100vh
. - Always have an
h1
on a webpage. Theh2
text on the left side could be theh1
. - The
.stat
selector could have usedul
element, since those are "list" of information about the user, or andl
element. - The text like
10k
314
shouldn't be heading element, since it doesn't really add any extra information if it were a heading tag like this one. Just usep
tag on it.
Aside from those, just the bug when resizing. Still, great job on this one.
Marked as helpful