Social Links Component

Community feedback

• 

  Alex•3,130

  @Alex-Archer-IPosted 3 months ago

  Hey!

  Cool work, congrats =)

  A few tips if you don't mind =)

  Semantic stuff.

  • It is preferable if the h tags follow the hierarchy - so after h1 better to use h2. - - - The links here is a list, so you can put them in the ul and li tags.
  • And every page should contain main tag, but since this project could be part of the bigger page, maybe it's not really necessary here =)

  CSS things.

  • You can more easier center the content by adding flex to the body.
  • I recommend you to use rem for font sizes. It's a special values which depends on the browser font settings.
  • Don't use percentages for border-radius. This property is better keep static.

  Hope that helps! Overall it is great work, keep doing =)

  Marked as helpful

  1
• 

  xcinek1337•70

  @xcinek1337Posted 3 months ago

  need to center

  0

  ---

  

  Dan Harabagiu-Dimitrescu•260

  @danHD2Posted 3 months ago

  @xcinek1337 how exactly?

  0