social link profile using CSS flex box

Community feedback

• 

  Agustina•155

  @aguscorvoPosted 3 months ago

  Congrats on solving the challenge! Here are some suggestions or tips:

  • Include semantic HTML tags like main, section, and article when necessary, instead of using divs for everything.
  • There’s something missing in the hover state: while the background color changes, the text color stays white, which makes it hard to read. According to the designs, it should be black.
  • From my understanding, "GitHub", "Frontend Mentor", etc, should be buttons.
  • Explore other ways to manage spacing between elements instead of using <br> tags. For example, you can use the gap property with flexbox.
  • Please check the size of the avatar image, the card size and the padding. Also, note that they may change slightly in mobile view.

  Happy coding! :)

  0