Responsive Social links profile made using HTML & vanilla CSS

Community feedback

• 

  Adann Jacinto•790

  @DanCodeCraftPosted 8 months ago

  Hey @Choconaldo, Nice work!

  I would suggest, for improvement, a few points:

  • For accessibility, do not use font size in pixels. Use rem instead.
  • You can add a transition effect when hovering the buttons. It will take you one line of code for a super nice effect.
  • A modern CSS reset could become a good habit of yours, saving lots of lines as well, keeping your code cleaner.

  Keep up the good work!

  Marked as helpful

  0

  ---

  

  P

  Leonardo Mendes•70

  @ChoconaldoPosted 8 months ago

  Thank you very much for your feedback, @DanCodeCraft .

  The rem suggestion is very good indeed. I'll do that in my next project.

  I need to study more about transitions. Good tip.

  About the reset, it seems that there is a ton of different types of it around the web and I simply can't find one generic enough to be used in every case. Any suggestions about that?

  Thanks a lota again.

  0

  

  Adann Jacinto•790

  @DanCodeCraftPosted 8 months ago

  @Choconaldo this is my absolute favorite one: https://www.joshwcomeau.com/css/custom-css-reset/

  It was suggested by Kevin Powell (check him out on YouTube in case you don't him. He is a wizard in frontend!). In the link you can see the full reset, you can copy and paste. And he also breaks down each section, so it's also a good reading.

  To not become a massive file, I suggest you to create a second CSS file, put the reset there and link it to your HTML file (you can check my recent projects in this site to have a visual idea if needed).

  Cheers :)

  Marked as helpful

  0

  

  P

  Leonardo Mendes•70

  @ChoconaldoPosted 8 months ago

  Thank you very much, @DanCodeCraft . I simply adore Mr. Powell. He's a genius!!

  1