Social Media Button Components

Community feedback

• 

  Paulo Vitor Ferreira Gonçalves Pivotto•40

  @high-rolls

  Posted 26 days ago

  Nice job! It looks very good, and it's a nice touch that you included your own social links already.

  You got the sizes correct for the most part, but forgot about add padding to the card element and some margin between the user summary and the links. That makes your solution look a bit more crammed than the intended design.

  Your solution is also using the browser's default font (Times New Roman in my case). That not only is different from the type of the design font (serif instead of sans-serif), but the default font varies wildly between user devices, sometimes even breaking the layouts.

  I also would follow the hint from the CSS report and not set the font-size attribute in pixels. To make the whole page default to "14px" use "875rem" instead, so that it scales when users change their browser's font size.

  00