social-links-profile

Community feedback

• 

  Dylan de Bruijn•3,230

  @DylandeBruijnPosted 4 months ago

  Hi @trskldnt,

  Great job on another finished project, you're on a roll! Nice use of CSS variables and proper class names.

  Some friendly constructive feedback:

  • Instead of using height try using min-height or just padding for the buttons. When you set the height of a button to a fixed value you get overflow issues when the text of the button gets too large. You can see this problem as well when you make the viewport really small.

  • Try experimenting with relative units like rem and em. They help you make your solution responsive. More information here.

  • Instead of using border-radius: 999px; to make the image round you could use border-radius: 50%. Just for your interest.

  • Instead of using row-gap you can just use gap. If I remember correctly row-gap is used in combination with grid if you just want to set the gap on the rows and not the columns.

  I hope you find my feedback helpful and if you have any other questions ask away!

  Marked as helpful

  1
• 

  FireDragonSrc•60

  @FireDragonSrcPosted 4 months ago

  Objetivo: Criar uma página web simples utilizando HTML e CSS

  0