@nainsworth
Posted
Everything looked very well structured, readable, and organized.
Here's a couple of things I would add/change:
- Would make HTML have a height of 100vh, change the body from grid to flex and center justify-content and align-items, and remove the 50px margin from the body. This would center your project to any device screen.
- Adding css variables to :root, would allow your reused set colors and other styles consistently and make things much easier for readability and usability.
Overall great work! Keep it up!
1
jovic-djordje• 150
@jovic-djordje
Posted
@nainsworth Thanks for leaving comment, I will try to apply this in next challenges.
0
