Darkstar• 1,000
@DarkstarXDD
Posted
Looks good. Couple of suggestions.
- I don't think the user's name and the verification status are headings. Definitely not
<h1>
s. You should only have one<h1>
heading for a page, which will be the primary text that explains the content of the page. It's better to have the user name and the status as<p>
elements. - I would always use
rem
when specifying themax-width
and avoid using percentage values formax-width
. - Is there a need for the
grid-auto-rows: minmax(10rem, auto)
?
Marked as helpful
0
Robel T. Hawelti• 100
@Rapbit27
Posted
@DarkstarXDD Thanks for the feedback!! I'll make sure to apply them in future projects too.
- As for the grid-auto-rows: minmax(10rem, auto) it was there when I was experimenting with the grid and forgot to remove it at the end.
1