Social links profile

Community feedback

• 

  P

  KonradJam•430

  @KonradJamPosted about 1 month ago

  Good job!

  I suggest a few small changes to your project:

  • If you want to center your component using Flexbox, you should set the .section height to 100vh and add align-items: center. This way, the margin won't be needed anymore.
  • For the links, it would be better to use an <a> tag with hover and focus effects.
  • Start using relative units. This will improve accessibility for users who have difficulty reading small text and have set their default browser font size larger than 16px.

  Marked as helpful

  0

  ---

  

  Zahra Abellou•250

  @zahraabelluPosted about 1 month ago

  @KonradJam Thanks for your feedback٫ It was helpful.

  0