All comments

• 

  Jordan Davenport•970

  @jjdavenportSubmitted 9 months ago

  What are you most proud of, and what would you do differently next time?

  I would use Flexbox for this layout, I would probably avoid using Grid for my ul next time.

  What challenges did you encounter, and how did you overcome them?

  Main challenge was overflow with mobile devices.

  What specific areas of your project would you like help with?

  Making a responsive site that works with all devices.

  

  Social links profile with flexbox

  1

  80
  

  P

  BuraYu•50

  @BuraYuPosted 9 months ago

  Hey Jordan,

  your project looks good!

  These would be my suggestions:

  -Images should be in the HTML not in the CSS (only background images). This way you could add alt text

  -In the CSS, you could add a <generic-name> to the font family. A fallback in case the main font doesn't work.

  -all the HTML elements are at the beginning of the row. This makes the code harder to read.

  All in all looks very good!

  Marked as helpful

  1

  This comment has received 2 repliesShow