Social links profile

Community feedback

• 

  P

  beowulf1958•1,450

  @beowulf1958Posted about 1 month ago

  Great job completing this challenge. It looks good at 1440px screen size. I loved that you customized it for you; that shows you are creative.

  Here are some suggestions to improve. First remember the DRY principle: don't repeat yourself. You declared font-family: 'Comfortaa'; in three different places. If you set the body element to font-family: 'Comfortaa'; the cascade will do the rest.

  It is not a good practice to position everything absolutely. You can, in fact, remove all the position, top, and left styles and the card will look just fine. Just add gap: 15px; to the .card style, and the card is good to go.

  Once the top and left styles are removed, .instagram, .github, and .front classes can be removed and replaced with a single .link class. This will save you some typing, be easier to maintain, and still look great.

  In order to center the card, best practice is to use flexbox or grid on the parent element (the body). This article will explain all the details. This will solve the responsiveness problem.

  Hope this helps.

  0
• 

  M Asim Saeed•270

  @asimsaeed353Posted about 1 month ago

  Great design impressive improvisation. You can apply padding on link elements to maximize their width. And apply text-decoration : none; and give them colors according to your design.

  0
• 

  foo-lin-xuan•30

  @foo-lin-xuanPosted about 1 month ago

  Great that you are having fun!

  0
• 

  rian•60

  @RIANdetwicePosted about 1 month ago

  I think the colors are too sharp

  0