using flexbox

Community feedback

• 

  

  ApplePieGiraffe•30,525

  @ApplePieGiraffe

  Posted almost 5 years ago

  Looks good, esraagamal!

  Just one small thing—some of the company logos near the bottom of the page look a little vertically stretched. Since you used flexbox to layout those logos, I'd suggest setting the align-items property to center rather than stretch, and think that might fix the issue.

  Otherwise, good job! 🎉

  Keep coding (and happy coding, too)!

  11
• 

  

  Matt Studdert•13,611

  @mattstuddert

  Posted almost 5 years ago

  Awesome work on this challenge, your solution looks great! @ApplePieGiraffe has made a great suggestion with the company logos. I'd also recommend removing the changing width when the buttons are hovered. It's not a common practice. Instead, I'd just keep the nice background-color and other styling changes that you've got.

  Your solution scales down well to mobile. Have you ever tried using min-width media queries instead of max-width? It's quite a common workflow with front-end developers to use them and work mobile-first. It can often lead to less CSS code and has the benefit of loading in fewer styles for mobile users, which can be a nice performance gain.

  I hope that helps. Keep up the great work! 👍

  01