@natashapl
Posted
Hi Deren! You did a great job for a first responsive solution. The only feedback I have is that there seems to be an excessive amount of margin around the section
and attribution
. I can tell you were trying to match the design, but it may be best to vertically center the section. Since you're using CSS grid, you can apply the following CSS rule to the main
tag:
.main { display: grid; height: 100vh; align-items: center; }
As for the attribution, consider decreasing the margin some, unless your intention is to hide it below the viewport.
I hope this is helpful. :)
Marked as helpful
@MorbitDemon
Posted
@natashapl That is indeed very helpful, thank you so much Natasha :3 !!
@natashapl
Posted
@MorbitDemon You're very welcome! :)