Social Links Profile

Community feedback

• 

  Nicholas Ainsworth•50

  @nainsworthPosted 5 months ago

  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•220

  @jovic-djordjePosted 5 months ago

  @nainsworth Thanks for leaving comment, I will try to apply this in next challenges.

  0