All comments

• 

  Celina Garcés•40

  @celinagrcsSubmitted 10 months ago

  

  social-links-profile-main

  2

  70
  

  Yousef Zamanpour•230

  @YousefZamanpourPosted 10 months ago

  Hi celina o/ Suggestion-1: it's always a good idea to use rem unit instead of pixel for font-sizes, and every 1rem is equal to 16px, so in the style guide provided to you it's mentioned to use 13px for font-size, so 13 / 16 = .8125 rem. Suggestion-2: usually when you have fixed width and height on your elements you eventually run into problem. in your css .social-links-container you have set width of 350px which is a good size, but instead use max-width. and remove the height in total or set it to min-height. overall great work

  Marked as helpful

  0