Social Links Profile

Community feedback

• 

  hitmorecode•6,680

  @hitmorecodePosted 13 days ago

  Nice well done. Just a few tips

  • Change the width of the card for smaller screens
  • The buttons should be links to those pages. Change the <p> tags to <a> tags or you can add the anchor element inside the <p> tags. You already have class button on the p tags, just change all p's for a's and don't forget to set text-decoration: none in your CSS.
  • To make the hover effect smoother add transition the the element and set it to 100ms or 300ms with ease-in-out

  👌👍

  Marked as helpful

  0
• 

  Natali 👻 Grimm•1,150

  @Grimm-NPosted 14 days ago

  Great job! U absolutely did it! I have one little suggestion for mobile version:

  It would be nice if the card didn’t stick directly to the edges of the window and had a small padding or margin. For example, setting the container size to 90% or using vh units could work well. This way, the card would adjust to the screen size and won’t cling to the window’s sides.

  Marked as helpful

  0
• 

  Zeke Achas•70

  @zekeer21Posted 14 days ago

  Your made it almost identical to the design. I can't say nothing. Great job.

  1